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
fix(checks): make am_pm check case sensitive #620
Conversation
Sorry for the close/reopen; I momentarily wanted nothing to do with this project after getting shit on in one of the issue threads. |
1e67e30
to
e6b23cd
Compare
Codecov Report
@@ Coverage Diff @@
## main #620 +/- ##
=======================================
Coverage 94.34% 94.34%
=======================================
Files 82 82
Lines 1131 1131
=======================================
Hits 1067 1067
Misses 64 64
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
e6b23cd
to
6502c97
Compare
6502c97
to
b2d1d26
Compare
Good fix, thanks for the PR - and I'm sorry for whatever happened in that issue all that time ago. |
Upon using proselint on an email, I immediately noticed that "lowercase am/pm" was not checking that am/pm was actually lower case.
I attempted to fix that here (and throw in some tests). But, I'm not a python hacker, and I have no idea what the usual workflows are for python projects. I did run
nosetests
within the tests directory, but my new test still fails. Is there some build step I'm missing? Or is my test broken?P.S. It would be great if you could throw a few words into the contributing guide about how to approach the project as an absolute Python beginner. E.g. is
pip install --user --upgrade .
the right way to install a modified version? Or, how do I run it without even installing it locally? :)