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

Allow reporting annotations without creating a new check #5

Closed
nielsvanvelzen opened this issue Dec 12, 2020 · 2 comments
Closed

Allow reporting annotations without creating a new check #5

nielsvanvelzen opened this issue Dec 12, 2020 · 2 comments

Comments

@nielsvanvelzen
Copy link

Right now this action adds a new check to report all annotations. Ideally I'd want to disable this behavior and just report annotations "normally" like other actions do.

For example, one of the linters I use is markdownlint and I enabled a problem matcher for that to create annotations. Another one is yutailang0119/action-android-lint that uses the GitHub actions toolkit to create the annotations from a Android XML file.

@jwgmeligmeyling
Copy link
Owner

jwgmeligmeyling commented Dec 12, 2020

Currently the Github API only allows pushing annotations by creating a new check run. I have asked Github to provide alternative APIs but as of now I have no clue how this could be done. More or less related to #2.

PRs are welcome :)

@nielsvanvelzen
Copy link
Author

I took a better look at the android linter I mentioned and it simply said it prints the annotations to stdout via the command helper from the toolkit: https://github.com/actions/toolkit/blob/main/packages/core/src/command.ts

I'll see if I can make a PR for it later

srini-hv pushed a commit to srini-hv/checkstyle-github-action that referenced this issue Jul 15, 2021
ChristopheClermont pushed a commit to ChristopheClermont/checkstyle-github-action that referenced this issue Apr 3, 2024
…_yarn/jest-circus-29.7.0

Bump jest-circus from 27.2.0 to 29.7.0
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 a pull request may close this issue.

2 participants