This repository was archived by the owner on Jan 10, 2023. It is now read-only.
Add documentation, Makefile, and linting#170
Open
Uh oh!
There was an error while loading. Please reload this page.
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Documentation and linting More linting and documentation usb_exceptions.py and the sign_*.py files pass linting checks More linting and documentation More linting and documentation pylint: ignore too-few-public-methods Spacing Linting Linting Documentation Linting Create .readthedocs.yml Create requirements.txt Add documentation link to README Update README.rst Update README.rst Update README.rst Update .gitignore Add call graphs pylint and flake8 pass Documentation Documentation fixes Add my Travis CI and Coveralls badges (#2) Add some tables of contents to the documentation Add linting to CI (#3) * Add linting to CI * Install flake8 and pylint Hide the original repo's badges (for now) (#4) Finish documentation skeletons for the sign_* files and usb_exceptions.py (#5) Finish documentation skeletons for fastboot.py and fastboot_debug.py (#6) Almost done with documentation skeletons Finish documentation skeletons Add contents to module docstrings Working on documentation contents Finish adding contents to module docstrings Replace sections with rubrics Document the sign_* modules and usb_exceptions.py Remove whitespace Fix typo Add tests for the signers Change 'ValueError' to more general 'Exception' Added tests for the 'GetPublicKey()' methods Add ADB keygen functionality (from @joeleong) Fix sign_* tests Add debugging code, work on documentation of filesync_protocol.py Working on documentation Remove debugging code Fill in some documentation Documentation and code formatting Add some documentation More documentation Put command onto one line Linting Linting Update documentation index.rst Remove CHECKLIST.md
ContributorAuthor
JeffLIrion commented Oct 19, 2019
I know that this pull request is pretty extensive and difficult to review. To help with this, I created two branches on my fork:
Here is a pull request from the latter to the former: https://github.com/JeffLIrion/python-adb/pull/7/files If you look through the changes, you'll see that there are no breaking changes. |
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
There shouldn't be any breaking changes here, but this will create merge conflicts with most (if not all) outstanding pull requests.