Skip to content

Auto format all files with configlet - #619

Merged
mk-mxp merged 1 commit into
exercism:mainfrom
mk-mxp:auto-format
Feb 19, 2024
Merged

Auto format all files with configlet#619
mk-mxp merged 1 commit into
exercism:mainfrom
mk-mxp:auto-format

Conversation

@mk-mxp

Copy link
Copy Markdown
Contributor

Now it should not do that when creating a new exercise.

@mk-mxpmk-mxp self-assigned this Feb 18, 2024
@mk-mxpmk-mxp added x:action/improve Improve existing functionality/content x:knowledge/none No existing Exercism knowledge required x:rep/tiny Tiny amount of reputation labels Feb 18, 2024
@mk-mxpmk-mxp mentioned this pull request Feb 18, 2024
30 tasks
@homersimpsons

homersimpsons commented Feb 18, 2024

Copy link
Copy Markdown
Contributor

Of course this is not really reviewable, with a quick glance to all changes it looks fine though.

@mk-mxp Can we add any CI job to ensure this formating is kept in next Pull Requests ?

@mk-mxp

Copy link
Copy Markdown
ContributorAuthor

@mk-mxp Can we add any CI job to ensure this formating is kept in next Pull Requests ?

I think we need to coordinate this with exercism officials. The configlet workflow is owned by them, and changes are made by a bot.

@mk-mxpmk-mxp mentioned this pull request Feb 19, 2024
5 tasks
@mk-mxp
mk-mxp merged commit be7dfcd into exercism:mainFeb 19, 2024
@mk-mxp
mk-mxp deleted the auto-format branch February 19, 2024 10:25
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

x:action/improveImprove existing functionality/contentx:knowledge/noneNo existing Exercism knowledge requiredx:rep/tinyTiny amount of reputation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@mk-mxp@homersimpsons