-
Notifications
You must be signed in to change notification settings - Fork 696
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
chore(cspell): merge spell-check-partial
and spell-check-all
#6596
Conversation
Signed-off-by: Kotaro Yoshimoto <pythagora.yoshimoto@gmail.com>
Signed-off-by: Kotaro Yoshimoto <pythagora.yoshimoto@gmail.com>
Signed-off-by: Kotaro Yoshimoto <pythagora.yoshimoto@gmail.com>
Signed-off-by: Kotaro Yoshimoto <pythagora.yoshimoto@gmail.com>
Signed-off-by: Kotaro Yoshimoto <pythagora.yoshimoto@gmail.com>
Signed-off-by: Kotaro Yoshimoto <pythagora.yoshimoto@gmail.com>
Signed-off-by: Kotaro Yoshimoto <pythagora.yoshimoto@gmail.com>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #6596 +/- ##
=======================================
Coverage 14.80% 14.80%
=======================================
Files 1915 1915
Lines 132301 132244 -57
Branches 39320 39316 -4
=======================================
- Hits 19581 19578 -3
+ Misses 90888 90832 -56
- Partials 21832 21834 +2
*This pull request uses carry forward flags. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@HansRobo How about adding workflow dispatch to github actions? |
Signed-off-by: Kotaro Yoshimoto <pythagora.yoshimoto@gmail.com>
@manato Could you review the change for bytetrack? |
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 regarding the bytetrack package! I appreciate your contribution!
…owarefoundation#6596) * chore(ci): expand the scope of spell-check-partial Signed-off-by: Kotaro Yoshimoto <pythagora.yoshimoto@gmail.com> * chore(cspell): clean up .cspell-partial.json Signed-off-by: Kotaro Yoshimoto <pythagora.yoshimoto@gmail.com> * chore(cspell): ignore words from bytetrack Signed-off-by: Kotaro Yoshimoto <pythagora.yoshimoto@gmail.com> * chore(cspell): rename spell-check-partial to spell-check Signed-off-by: Kotaro Yoshimoto <pythagora.yoshimoto@gmail.com> * chore(cspell): delete spell-check-all Signed-off-by: Kotaro Yoshimoto <pythagora.yoshimoto@gmail.com> * chore(cspell): leave spell-check-partial name for CI required settings Signed-off-by: Kotaro Yoshimoto <pythagora.yoshimoto@gmail.com> * chore(cspell): restore job name in spell-check-partial.yaml Signed-off-by: Kotaro Yoshimoto <pythagora.yoshimoto@gmail.com> * chore(cspell): add workflow dispatch trigger to spell-check-partial Signed-off-by: Kotaro Yoshimoto <pythagora.yoshimoto@gmail.com> --------- Signed-off-by: Kotaro Yoshimoto <pythagora.yoshimoto@gmail.com> Signed-off-by: kaigohirao <kaigo.hirao@proxima-ai-tech.com>
…owarefoundation#6596) * chore(ci): expand the scope of spell-check-partial Signed-off-by: Kotaro Yoshimoto <pythagora.yoshimoto@gmail.com> * chore(cspell): clean up .cspell-partial.json Signed-off-by: Kotaro Yoshimoto <pythagora.yoshimoto@gmail.com> * chore(cspell): ignore words from bytetrack Signed-off-by: Kotaro Yoshimoto <pythagora.yoshimoto@gmail.com> * chore(cspell): rename spell-check-partial to spell-check Signed-off-by: Kotaro Yoshimoto <pythagora.yoshimoto@gmail.com> * chore(cspell): delete spell-check-all Signed-off-by: Kotaro Yoshimoto <pythagora.yoshimoto@gmail.com> * chore(cspell): leave spell-check-partial name for CI required settings Signed-off-by: Kotaro Yoshimoto <pythagora.yoshimoto@gmail.com> * chore(cspell): restore job name in spell-check-partial.yaml Signed-off-by: Kotaro Yoshimoto <pythagora.yoshimoto@gmail.com> * chore(cspell): add workflow dispatch trigger to spell-check-partial Signed-off-by: Kotaro Yoshimoto <pythagora.yoshimoto@gmail.com> --------- Signed-off-by: Kotaro Yoshimoto <pythagora.yoshimoto@gmail.com>
Description
There are two spell-check workflows now.
One is
spell-check-all
to check whole code of autoware.universe to observe all cspell errors.The another is
spell-check-partial
to prevent additional typos from being introduced.These two workflows were used differently, but now the differences are gone, so they are combined in this pull-request.
Note: This pull-request don't edit the workflow name,
spell-check-partial
, because the name is reffered by GitHub settings.Tests performed
The
spell-check-partial
worflow passed.Effects on system behavior
Not applicable.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.
In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.
Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.
After all checkboxes are checked, anyone who has write access can merge the PR.