-
-
Notifications
You must be signed in to change notification settings - Fork 949
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
fix: format check stage in the CI #2737
Conversation
@@ -83,8 +83,6 @@ jobs: | |||
uses: sonatype-nexus-community/nancy-github-action@v1.0.2 | |||
- run: npm install | |||
name: Install node deps | |||
- name: Run prettier |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is the formatting now checked somewhere else?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can also rollback 20fdfe8 if the format
check needs to be kept as a pre-req for tests
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Take a look now please
Codecov Report
@@ Coverage Diff @@
## master #2737 +/- ##
==========================================
+ Coverage 75.15% 75.19% +0.04%
==========================================
Files 294 294
Lines 17068 17068
==========================================
+ Hits 12828 12835 +7
+ Misses 3264 3258 -6
+ Partials 976 975 -1
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
ac46cb9
to
ed305a0
Compare
This PR fixes the CI workflow. The format checks had been moved to an independent workflow (https://github.com/ory/kratos/blob/master/.github/workflows/format.yml) to be triggered for PR's and master builds.
Related issue(s)
Build failure on master
Checklist
introduces a new feature.
contributing code guidelines.
vulnerability. If this pull request addresses a security. vulnerability, I
confirm that I got green light (please contact
security@ory.sh) from the maintainers to push
the changes.
works.
Further Comments