Skip to content

Conversation

timgates42
Copy link
Contributor

There are small typos in:

  • tests/frontend/viewport_toggle_controller_test.js
  • warehouse/email/ses/models.py
  • warehouse/legacy/api/json.py
  • warehouse/rate_limiting/init.py

Fixes:

  • Should read vulnerabilities rather than vulnerabilties.
  • Should read unverified rather than unverfied.
  • Should read starting rather than satarting.
  • Should read skip rather than skipp.

Semi-automated pull request generated by
https://github.com/timgates42/meticulous/blob/master/docs/NOTE.md

There are small typos in:
- tests/frontend/viewport_toggle_controller_test.js
- warehouse/email/ses/models.py
- warehouse/legacy/api/json.py
- warehouse/rate_limiting/__init__.py

Fixes:
- Should read `vulnerabilities` rather than `vulnerabilties`.
- Should read `unverified` rather than `unverfied`.
- Should read `starting` rather than `satarting`.
- Should read `skip` rather than `skipp`.

Signed-off-by: Tim Gates <[email protected]>
@timgates42 timgates42 requested a review from a team as a code owner July 19, 2022 10:58
@di
Copy link
Member

di commented Jul 19, 2022

Hi, are you able to include whatever tool detected these to our linting jobs as well?

@timgates42
Copy link
Contributor Author

I have tried this in the past with some repositories and its met with some opposition - It uses aspell which isn't a pure python linter and so some developers have been against it as it make local checking of their pull requests have a non python dependency - That said if that isn't an issue here I can add a PR for consideration.

@miketheman
Copy link
Member

Related: #11568

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants