Skip to content

Fix pre-commit cspell failure for EDF Go Electric tariff token - #4412

Merged
springfall2008 merged 3 commits into
mainfrom
copilot/fix-pre-commit-job
Aug 1, 2026
Merged

Fix pre-commit cspell failure for EDF Go Electric tariff token#4412
springfall2008 merged 3 commits into
mainfrom
copilot/fix-pre-commit-job

Conversation

Copilot AI commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

The code-quality / pre-commit (pull_request) job was failing on cspell due to an unrecognized tariff token introduced in EDF tariff catalogue updates. This PR resolves that spelling-gate failure without changing runtime behavior.

  • CI failure source

    • cspell flagged GOELEC in apps/predbat/tariff_catalogue.py (EDF product code segment).
  • Change made

    • Added GOELEC to the workspace custom dictionary:
      • .cspell/custom-dictionary-workspace.txt
  • Behavioral impact

    • No application logic changes; this is a spelling dictionary update only.
diff --git a/.cspell/custom-dictionary-workspace.txt b/.cspell/custom-dictionary-workspace.txt
@@
 Ginlong
+GOELEC
 GivEnergy

Copilot AI changed the title [WIP] Fix failing GitHub Actions job code-quality / pre-commit Fix pre-commit cspell failure for EDF Go Electric tariff token Aug 1, 2026
Copilot AI requested a review from springfall2008 August 1, 2026 15:42
@springfall2008
springfall2008 marked this pull request as ready for review August 1, 2026 16:22
Copilot AI review requested due to automatic review settings August 1, 2026 16:22

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the repository’s CSpell workspace dictionary to allow the EDF “GOELEC” tariff token so pre-commit spelling checks stop failing, without changing runtime behavior.

Changes:

  • Added GOELEC to .cspell/custom-dictionary-workspace.txt to satisfy cspell.
  • (Follow-up needed) Keep the dictionary’s alphabetical ordering so pre-commit doesn’t rewrite/fail the file.

Comment on lines 182 to 186
Ginlong
GOELEC
GivEnergy
givtcp
gnueabihf
@springfall2008
springfall2008 merged commit 72b1603 into main Aug 1, 2026
2 checks passed
@springfall2008
springfall2008 deleted the copilot/fix-pre-commit-job branch August 1, 2026 16:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants