-
Notifications
You must be signed in to change notification settings - Fork 31
New install #90
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
Merged
Merged
New install #90
Changes from all commits
Commits
Show all changes
19 commits
Select commit
Hold shift + click to select a range
66fae37
Add py-libp2p-style typecheck functionality and PR workflow
acul71 df2903a
Fix mypy type annotation errors
acul71 661fc64
Fix coverage target and update AUTHORS file
acul71 9242362
Compile release notes for v0.0.10
acul71 1191997
Update py-cid dependency to use specific git commit
acul71 eb14eb3
Resolve merge conflict: comment out master's py-cid and use specific β¦
acul71 1537961
doc: filter out .html generated files
acul71 a5a6b52
doc: restored v0.0.10 test release fragments, cleaned HISTORY.rst, adβ¦
acul71 382e34d
doc: chore authors
acul71 3e8aaf8
doc: updated contributing to current version
acul71 9e21bce
doc: updated README info
acul71 1a42966
doc: update licences and filtering
acul71 b3d4622
doc: chore
acul71 7986bab
fix: updated py-cid to pypi new package 0.3.1
acul71 9465164
update bumpversion locations
pacrob 6515f7d
raise current version to 0.0.10 based on tag and release on June 18
pacrob 653e69f
fix bumpversion config
pacrob 68c63bf
feat: add windows in ci/cd
acul71 66b7cfa
compress newsfragments
pacrob File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,4 +1,18 @@ | ||
| Steven Buss <[email protected]> | ||
| Alexander Schlarb <[email protected]> | ||
| Fred Thomsen <[email protected]> | ||
| Jesse Weinstein <[email protected]> | ||
| Alexander Schlarb <[email protected]> | ||
| mhchia | ||
| acul71 | ||
| arcinston | ||
| pacrob | ||
| raulk | ||
| robzajac | ||
| zixuanzh | ||
| hugovk | ||
| justheuristic | ||
| manusheel | ||
| rodvagg | ||
| venkat | ||
| fredthomsen | ||
| web3-bot |
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -20,6 +20,7 @@ Report bugs at https://github.com/multiformats/py-multiaddr/issues. | |
| If you are reporting a bug, please include: | ||
|
|
||
| * Your operating system name and version. | ||
| * Python version. | ||
| * Any details about your local setup that might be helpful in troubleshooting. | ||
| * Detailed steps to reproduce the bug. | ||
|
|
||
|
|
@@ -62,27 +63,33 @@ Ready to contribute? Here's how to set up `multiaddr` for local development. | |
| 1. Fork the `multiaddr` repo on GitHub. | ||
| 2. Clone your fork locally:: | ||
|
|
||
| $ git clone [email protected]:your_name_here/multiaddr.git | ||
| $ git clone [email protected]:your_name_here/py-multiaddr.git | ||
|
|
||
| 3. Install your local copy into a virtualenv. Assuming you have virtualenvwrapper installed, this is how you set up your fork for local development:: | ||
| 3. Install your local copy into a virtual environment:: | ||
|
|
||
| $ mkvirtualenv multiaddr | ||
| $ cd multiaddr/ | ||
| $ python setup.py develop | ||
| $ python -m venv venv | ||
| $ source venv/bin/activate # On Windows: venv\Scripts\activate | ||
| $ pip install -e ".[dev]" | ||
|
|
||
| 4. Create a branch for local development:: | ||
|
|
||
| $ git checkout -b name-of-your-bugfix-or-feature | ||
|
|
||
| Now you can make your changes locally. | ||
|
|
||
| 5. When you're done making changes, check that your changes pass flake8 and the tests, including testing other Python versions with tox:: | ||
| 5. When you're done making changes, run the development workflow:: | ||
|
|
||
| $ flake8 multiaddr tests | ||
| $ python setup.py test | ||
| $ tox | ||
| $ make pr | ||
|
|
||
| To get flake8 and tox, just pip install them into your virtualenv. | ||
| This will run: clean β fix β lint β typecheck β test | ||
|
|
||
| Or run individual commands:: | ||
|
|
||
| $ make fix # Fix formatting & linting issues with ruff | ||
| $ make lint # Run pre-commit hooks on all files | ||
| $ make typecheck # Run mypy and pyrefly type checking | ||
| $ make test # Run tests with pytest | ||
| $ make coverage # Run tests with coverage report | ||
|
|
||
| 6. Commit your changes and push your branch to GitHub:: | ||
|
|
||
|
|
@@ -101,13 +108,41 @@ Before you submit a pull request, check that it meets these guidelines: | |
| 2. If the pull request adds functionality, the docs should be updated. Put | ||
| your new functionality into a function with a docstring, and add the | ||
| feature to the list in README.rst. | ||
| 3. The pull request should work for Python 2.7, 3.4+ and PyPy3. Check | ||
| https://travis-ci.org/multiformats/py-multiaddr/pull_requests | ||
| and make sure that the tests pass for all supported Python versions. | ||
| 3. The pull request should work for Python 3.10+ (Python 3.9 support was dropped). | ||
| 4. All type checking must pass (mypy and pyrefly). | ||
| 5. All pre-commit hooks must pass. | ||
| 6. Code must be formatted with ruff. | ||
|
|
||
| Development Workflow | ||
| -------------------- | ||
|
|
||
| The project follows a py-libp2p-style development workflow: | ||
|
|
||
| 1. **Clean**: Remove build artifacts | ||
| 2. **Fix**: Auto-fix formatting and linting issues | ||
| 3. **Lint**: Run pre-commit hooks | ||
| 4. **Typecheck**: Run mypy and pyrefly | ||
| 5. **Test**: Run the test suite | ||
|
|
||
| Use ``make pr`` to run the complete workflow. | ||
|
|
||
| Release Notes | ||
| ------------- | ||
|
|
||
| When contributing, please add a newsfragment file in the ``newsfragments/`` directory. | ||
| See ``newsfragments/README.md`` for details on the format and types. | ||
|
|
||
| Tips | ||
| ---- | ||
|
|
||
| To run a subset of tests:: | ||
|
|
||
| $ python -m unittest tests.test_multiaddr | ||
| $ python -m pytest tests/test_multiaddr.py | ||
|
|
||
| To run with coverage:: | ||
|
|
||
| $ make coverage | ||
|
|
||
| To build documentation:: | ||
|
|
||
| $ make docs | ||
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,11 +1,15 @@ | ||
| include AUTHORS | ||
| include CONTRIBUTING.rst | ||
| include HISTORY.rst | ||
| include LICENSE | ||
| include LICENSE-APACHE2 | ||
| include LICENSE-MIT | ||
| include README.rst | ||
|
|
||
| recursive-include tests * | ||
|
|
||
| global-include *.pyi | ||
|
|
||
| recursive-exclude * __pycache__ | ||
| recursive-exclude * *.py[co] | ||
|
|
||
| recursive-include docs *.rst conf.py Makefile make.bat | ||
| prune .tox | ||
| prune venv* |
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
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1 +1,4 @@ | ||
| Contributing | ||
| ============ | ||
|
|
||
| .. include:: ../CONTRIBUTING.rst |
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -2,4 +2,4 @@ | |
|
|
||
| __author__ = "Steven Buss" | ||
| __email__ = "[email protected]" | ||
| __version__ = "0.0.9" | ||
| __version__ = "0.0.10" | ||
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.