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

Follow up changes to Runic formatting #1099

Merged
merged 1 commit into from
Oct 31, 2024
Merged

Follow up changes to Runic formatting #1099

merged 1 commit into from
Oct 31, 2024

Conversation

fredrikekre
Copy link
Member

@fredrikekre fredrikekre commented Oct 31, 2024

  • Add commit to blame ignore list
  • Mention Runic formatting in the CHANGELOG.md
  • Run Runic check on CI

Copy link

codecov bot commented Oct 31, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 93.57%. Comparing base (c50ce06) to head (7cf448e).
Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1099   +/-   ##
=======================================
  Coverage   93.57%   93.57%           
=======================================
  Files          39       39           
  Lines        6074     6074           
=======================================
  Hits         5684     5684           
  Misses        390      390           

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

@fredrikekre fredrikekre changed the title Add Runic formatting to i) blame ignore list and ii) changelog Follow up changes to Runic formatting Oct 31, 2024
@KnutAM
Copy link
Member

KnutAM commented Oct 31, 2024

Really nice with having Runic here!

I think some instructions for how to run locally (both as git hook and without) in CONTRIUBTING would be helpful.
Does it also make sense to add some extra config / scripts to facilitate that process?

@fredrikekre
Copy link
Member Author

Added some info to CONTRIBUTING.md, perhaps you can try it out?

Copy link
Member

@KnutAM KnutAM left a comment

Choose a reason for hiding this comment

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

Works for me with these suggested changes

CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Show resolved Hide resolved
 - Add commit to blame ignore list
 - Mention Runic formatting in the CHANGELOG.md
 - Run Runic check on CI
 - Add runic-pre-commit to pre-commit configuration
 - Add information about Runic and pre-commit to CONTRIBUTING.md
@fredrikekre fredrikekre merged commit b56e2c9 into master Oct 31, 2024
16 checks passed
@fredrikekre fredrikekre deleted the fe/runic-ignore branch October 31, 2024 22:19
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.

2 participants