-
Notifications
You must be signed in to change notification settings - Fork 73
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
[CI] Update cpp-linter ci (false file-annotations) #2464
Conversation
📝 TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #2464. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/. |
e60424c
to
0bc3ebe
Compare
0d378ed
to
def40ad
Compare
For now we use cpp-linter as option 'file-annotations' but there is too many annotations and that make hard to read code So make false file-annotations & replace that to step-summary - fix typo for check cpp-linter exec ref : https://github.com/marketplace/actions/c-c-linter **Changes proposed in this PR:** - modified: .github/workflows/cpp_linter.yml - modified: nntrainer/dataset/func_data_producer.h Resolves: - nnstreamer#2463 **Self evaluation:** 1. Build test: [X]Passed [ ]Failed [ ]Skipped 2. Run test: [X]Passed [ ]Failed [ ]Skipped Signed-off-by: Donghak PARK <[email protected]>
def40ad
to
0b644a3
Compare
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.
nice work!
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.
@DonghakPark, 💯 All CI checkers are successfully verified. Thanks.
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.
LGTM
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.
LGTM!
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.
LGTM
For now we use cpp-linter as option 'file-annotations' but there is too many annotations and that make hard to read code
So make false file-annotations & replace that to step-summary
ref : https://github.com/marketplace/actions/c-c-linter
Changes proposed in this PR:
Resolves:
--> I will add nntrainer-review-bot that make comment in PR page.
--> for now you can check error/warning in action's summary page.
Self evaluation: