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

Fix reviewdog exit code bug and error format syntax problem #4

Merged
merged 8 commits into from
Nov 23, 2020
Merged

Fix reviewdog exit code bug and error format syntax problem #4

merged 8 commits into from
Nov 23, 2020

Conversation

rickstaa
Copy link
Member

@rickstaa rickstaa commented Nov 2, 2020

As explained in #3 the current GitHub actions don't seem to work. Apart from now throwing an error when a warning was generated by remark-lint it also specifies the wrong errorformat for reviewdog. As a result no error will be thrown and no warnings are displayed.

asbjornu and others added 4 commits July 9, 2020 10:50
The current errorformat was incorrect see #3. I fixed this by adding the
right syntax to the reviewdog pre-defined error formats. This syntax can
be found under the `remark-lint` tag.
@rickstaa
Copy link
Member Author

rickstaa commented Nov 2, 2020

This pull request is blocked by reviewdog/errorformat#79.

@rickstaa
Copy link
Member Author

rickstaa commented Nov 23, 2020

The fix has been merged with the review dog main branch. We can close this pull request as soon as review dog v0.11.0 is available.

This commit temporary switches to use the reviewdog nightly branch. We
have to revert back to using the stable branch after v0.12.0 is
released.
@prologic prologic merged commit 0e804be into reviewdog:master Nov 23, 2020
@rickstaa
Copy link
Member Author

rickstaa commented Dec 4, 2020

I pushed a fix to the reviewdog nightly branch reviewdog/nightly#6.

@rickstaa rickstaa deleted the fix_reviewdog_exit_code_bug branch December 4, 2020 08:53
@rickstaa rickstaa restored the fix_reviewdog_exit_code_bug branch December 5, 2020 14:56
@rickstaa
Copy link
Member Author

rickstaa commented Dec 5, 2020

The fix has been merged I opened a new pull request #5 to temporary switch to using the nightly branch until the new version has been released.

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