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.

Sign up for GitHub

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

Jump to bottom

Add configlet fmt to CI #622

Merged
merged 2 commits into from
Feb 21, 2024
Merged

Add configlet fmt to CI #622

merged 2 commits into from
Feb 21, 2024

Conversation

mk-mxp
Copy link
Contributor

@mk-mxp mk-mxp commented Feb 20, 2024

@mk-mxp mk-mxp self-assigned this Feb 20, 2024
@mk-mxp mk-mxp added x:action/improve Improve existing functionality/content x:knowledge/none No existing Exercism knowledge required x:type/ci Work on Continuous Integration (e.g. GitHub Actions workflows) x:size/tiny Tiny amount of work x:rep/tiny Tiny amount of reputation labels Feb 20, 2024
@mk-mxp mk-mxp requested a review from homersimpsons February 20, 2024 21:09
@mk-mxp mk-mxp force-pushed the check-json-fmt-in-ci branch from 75b45bc to 1755277 Compare February 21, 2024 18:16
@mk-mxp mk-mxp force-pushed the check-json-fmt-in-ci branch from 1755277 to 6c07106 Compare February 21, 2024 18:17
@mk-mxp
Copy link
Contributor Author

mk-mxp commented Feb 21, 2024

@homersimpsons FYI: The TOML is needed, too. So I added that and merged.

@mk-mxp mk-mxp merged commit 2b6b6d9 into exercism:main Feb 21, 2024
12 checks passed
@mk-mxp mk-mxp deleted the check-json-fmt-in-ci branch February 21, 2024 18:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@homersimpsons homersimpsons homersimpsons approved these changes

Assignees

@mk-mxp mk-mxp

Labels
x:action/improve Improve existing functionality/content x:knowledge/none No existing Exercism knowledge required x:rep/tiny Tiny amount of reputation x:size/tiny Tiny amount of work x:type/ci Work on Continuous Integration (e.g. GitHub Actions workflows)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mk-mxp @homersimpsons