-
-
Notifications
You must be signed in to change notification settings - Fork 279
fixes yaml example #1350
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
fixes yaml example #1350
Conversation
Hi, thanks for fixing the typo. we need to follow the commit rule to pass the CI. could you please modify it? thanks! |
Like this @Lee-W ? |
hmm... I am not sure what you changed. but I can take over and fix the commit issue. Thanks! |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1350 +/- ##
==========================================
+ Coverage 97.33% 97.55% +0.21%
==========================================
Files 42 55 +13
Lines 2104 2614 +510
==========================================
+ Hits 2048 2550 +502
- Misses 56 64 +8
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Thank you. I see, my commit message was still wrong! I'm new to this project and just tried and played around. I found these typos and just quickly fixed them using Githubs UI. I were unable to identify any contribution guidelines (either by CONTRIBUTING.md nor documentation) easily. @Lee-W are there any contribution guidelines highlighting this? |
No worries 🙂 Yep, it's in the docs https://commitizen-tools.github.io/commitizen/contributing/ |
Description
Fixes documentation typos.
Checklist
Add test cases to all the changes you introduceRun./scripts/format
and./scripts/test
locally to ensure this change passes linter check and testTest the changes on the local machine manuallyExpected behavior
n/A
Steps to Test This Pull Request
n/A
Additional context
n/A