Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Nyaa development helper (tool) #324

Merged
merged 2 commits into from Aug 5, 2017
Merged

Nyaa development helper (tool) #324

merged 2 commits into from Aug 5, 2017

Conversation

sharkykh
Copy link
Contributor

@sharkykh sharkykh commented Aug 1, 2017 •
edited

✔️ Tested.

Still not sure if I should merge fix/autolint and isort commands

  • Fix some more lint/import sorting issues
  • Add a new tool to replace lint.sh (only deprecated, still works):
Nyaa Development Helper
=======================

Usage: ./dev.py command [different arguments]
Command can be one of the following:

  lint | check       : do a lint check (flake8 + flake8-isort)
  fix  | autolint    : try and auto-fix lint (autopep8)
  isort              : fix import sorting (isort)
  test | pytest      : run tests (pytest)
  help | -h | --help : show this help and exit

You may pass different arguments to the script that is being run.
For example: dev.py test tests/ --verbose

@sharkykh sharkykh added this to In Progress in App Restructuring Aug 2, 2017
@sharkykh
Add new tool for developing (lint/autopep8/test)
ba16ae8 
New tool uses flake8 and isort for lint checks.
Deprecate existing tool (still works)
Update readme
Update Travis config
@sharkykh
Simplify checking config values
fba47fb 
note2self: Next time, read the docs.

Also add a small docstring, and change some texts
@sharkykh sharkykh merged commit 024c900 into master Aug 5, 2017
@sharkykh sharkykh deleted the new-dev-tool branch August 5, 2017 21:04
@sharkykh sharkykh moved this from In Progress to Done in App Restructuring Aug 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
App Restructuring
Done
Milestone
No milestone
1 participant
@sharkykh