Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use ignore words file with codespell #6821

Merged
merged 1 commit into from
Apr 11, 2024

Conversation

mkindahl
Copy link
Contributor

@mkindahl mkindahl commented Apr 11, 2024

It is not possible to automatically backport pull requests that makes modifications to workflow files and since the codespell action has a hard-coded list of ignored words as options, this means that any changes to the ignored codespell words cannot be backported.

This pull request fixes this by using a ignore words file instead, which means that adding new words does not require changing a workflow file and hence the pull requests can be automatically backported.

Disable-check: force-changelog-file

@mkindahl mkindahl self-assigned this Apr 11, 2024
@mkindahl mkindahl force-pushed the codespell-ignore-words-file branch from 27765d3 to cb01cfc Compare April 11, 2024 06:38
.github/ignore-words Outdated Show resolved Hide resolved
@mkindahl mkindahl force-pushed the codespell-ignore-words-file branch from cb01cfc to 5394ce4 Compare April 11, 2024 06:42
It is not possible to automatically backport pull requests that makes
modifications to workflow files and since the codespell action has a
hard-coded list of ignored words as options, this means that any
changes to the ignored codespell words cannot be backported.

This pull request fixes this by using a ignore words file instead,
which means that adding new words does not require changing a workflow
file and hence the pull requests can be automatically backported.
@mkindahl mkindahl force-pushed the codespell-ignore-words-file branch from 5394ce4 to abe9a4c Compare April 11, 2024 06:42
Copy link

codecov bot commented Apr 11, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 80.91%. Comparing base (59f50f2) to head (abe9a4c).
Report is 104 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6821      +/-   ##
==========================================
+ Coverage   80.06%   80.91%   +0.84%     
==========================================
  Files         190      194       +4     
  Lines       37181    36927     -254     
  Branches     9450     9669     +219     
==========================================
+ Hits        29770    29878     +108     
- Misses       2997     3183     +186     
+ Partials     4414     3866     -548     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@mkindahl mkindahl merged commit 971e6c3 into timescale:main Apr 11, 2024
41 of 42 checks passed
@mkindahl mkindahl deleted the codespell-ignore-words-file branch April 11, 2024 08:51
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