Conversation
javierjulio
force-pushed
the
typos-workflow
branch
from
912b2ab to
6148680
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #8713 +/- ##
=======================================
Coverage 99.11% 99.11%
=======================================
Files 141 141
Lines 4076 4076
=======================================
Hits 4040 4040
Misses 36 36 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
| column :title | ||
| column(:author) { |post| post.author.full_name } | ||
| column('bODY', humanize_name: false) # preserve case | ||
| column('body', humanize_name: false) # preserves case of column title |
There was a problem hiding this comment.
@javierjulio, this was a fair use of 'bODY'. The "typo" + the comment shows that you can customize the title the way you need. The new comment looks good, I'm just talking about leaving 'bODY' as an exception of the new flow.
There was a problem hiding this comment.
@mgrunberg yes, I know but I was cheating because its all lowercase and by default will capitalize so I think that was sufficient. Let me look into another update to ignore this specific word in only this file.
2 participants
This adds a workflow to run typos CLI using Reviewdog and includes a
_typos.tomlconfiguration to address false positives. The "succesfully_destroyed" typo is ignored here so the workflow will go green as the typo correction will be addressed in #8712.