Comments (10)
I'm am, too, interested in this. Would you mind if I submit a pull request for this?
from reviewdog.
Sure. Do you have some rough design or something?
from reviewdog.
@haya14busa Did migrating to a checker fix this issue?
from reviewdog.
yes. For check API, we don't need to care about this problem.
reviewdog is still supporting reports through github review comments, so the problem isn't solved yet for -reporter=github-pr-review.
but i'll close this issue for now because i don't have plan to work on this issue. maybe we can re-visit later.
Thanks :)
from reviewdog.
👍 Just curious, any particular reason to keep the old reviewer? Since the checks API seems better in every single way, maybe the old could be removed?
from reviewdog.
New checks API is generally better, but the API is only available for GitHub apps.
So, reviewdog CLI need to talk to reviewdog server I created and the server will post Check API to GitHub.
As for -reporter=github-pr-review feature, reviewdog CLI directly talks to GitHub with personal access token.
For private repo users who don't want to, or it's difficult to trust my reviewdog server, it's better to use github-pr-review feature because it works standalone.
As for OSS repository, I think users should choose github-pr-check reporter by default unless the reviewdog server is broken ;)
from reviewdog.
For private repo users who don't want to, or it's difficult to trust my reviewdog server, it's better to use github-pr-review feature because it works standalone.
@haya14busa Ok. In that case, this issue is still standing for private repos. Would you mind reopening it (and maybe modify description that it's only relevant for private repos)? Or would you like me to create a new issue?
Do you have some rough design or something?
Idea: 1) find the first comment by its own user and 2a) if found, modify the comment; otherwise 2b) post a new comment. This is what CodeCov does. See here for an example (click "Edited" in the comment header). What do you think of this approach?
from reviewdog.
Would you mind reopening it (and maybe modify description that it's only relevant for private repos)?
@haya14busa Ping.
from reviewdog.
This may fixed one reviedog supports GitHub actions
from reviewdog.
The latest reviewdog supports GitHub Action and can use github-pr-check reporter.
from reviewdog.
Related Issues (20)
- 'github-pr-review' reporter is not making suggestion for blank line addition
- Support for Vale text linter HOT 2
- Bundle related suggestions together
- [BUG] Sarif format not working HOT 9
- fail to run reviewdog: XML syntax error on line 10: expected attribute name in element HOT 1
- 'reviewdog: .git exist but is not a directory' in submodule
- Add support for Azure Devops
- I want to konw which LLM do you uesd HOT 4
- File filter for github doesn't correctly submit comments HOT 3
- Wrong line number is sent to Bitbucket Server Code Insight Reports HOT 2
- Feature Request: Option to Disable the `"reported by reviewdog 🐶"` Prefix in Review Messages HOT 11
- New GitHub Action: terraform-validate HOT 2
- gitlab-mr-discussion fail to get diff HOT 3
- [BUG] breaks if file path contains parenthesis
- There was a lint problem running locally, but when I ran reviewdog on CI, there was no comment output HOT 1
- Logo Visibility Issue in GitHub Dark Theme
- GitHub Pull Request diff API responds with 406 — diff too large HOT 17
- github-pr-review reporter: Marks review threads as resolved if that are resolved
- ESLint - Reviewdog output missing file names
- Reviewdog reports issues in old lines instead of new ones
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from reviewdog.