Skip to content

Commit

Permalink
Merge pull request #29 from jannikmi/fix-missing-classes-folder
Browse files Browse the repository at this point in the history
poetry based packaging
  • Loading branch information
jannikmi authored Jun 15, 2022
2 parents 69fe7da + 319fd28 commit 9d72943
Show file tree
Hide file tree
Showing 13 changed files with 134 additions and 363 deletions.
9 changes: 6 additions & 3 deletions .github/workflows/build_test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -55,14 +55,17 @@ jobs:
with:
python-version: 3.7

- name: Install build dependencies
run: |
pip install poetry build
- name: Fetch version
id: fetch_version
run: echo "::set-output name=version_nr::$(cat VERSION)"
run: echo "::set-output name=version_nr::$(poetry version -s)"

- name: Build a binary wheel and a source tarball
run: |
pip install setuptools wheel
python setup.py sdist bdist_wheel --python-tag py37.py38.py39.py310
python -m build
- name: Create Release
id: create_release
Expand Down
16 changes: 8 additions & 8 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -10,11 +10,11 @@ repos:
- id: check-added-large-files
- id: check-case-conflict
- id: check-merge-conflict
# - id: name-tests-test
# - id: name-tests-test
- id: check-docstring-first
- id: requirements-txt-fixer
- id: detect-private-key
# - id: check-executables-have-shebangs
# - id: check-executables-have-shebangs

- repo: https://github.com/pycqa/isort
rev: 5.10.1
Expand Down Expand Up @@ -51,12 +51,12 @@ repos:
- flake8-comprehensions
- flake8-tidy-imports

# - repo: https://github.com/mgedmin/check-manifest
# rev: "0.47"
# hooks:
# - id: check-manifest
# args: [ "--no-build-isolation", "--ignore", "*.png,.travis/*,docs/*,paper/*,build_n_install.py,publish.py,readthedocs.yml" ]
# additional_dependencies: [ numpy ]
- repo: https://github.com/mgedmin/check-manifest
rev: "0.48"
hooks:
- id: check-manifest
args: [ "--no-build-isolation", "--ignore", "*.png,docs/*,paper/*,setup.py,publish.py,readthedocs.yml,poetry.lock" ]
additional_dependencies: [ numpy, poetry==1.1.11 ]

# TODO sphinx check
# - repo: https://github.com/myint/rstcheck
Expand Down
51 changes: 0 additions & 51 deletions .travis.yml

This file was deleted.

6 changes: 6 additions & 0 deletions CHANGELOG.rst
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ Changelog

TODOs

* build html docs and include with package: "docs/_build/html/*"
* run speed and numerical tests with the new C evaluation method!
* Improve tests
* compare poly.num_ops of different factorisations. tests?
Expand Down Expand Up @@ -53,6 +54,11 @@ idea:
- the evaluation of subtrees is independent and could theoretically be done in parallel
probably not worth the effort. more reasonable to just evaluate multiple polynomials in parallel

3.0.3 (2022-06-15)
__________________

* bugfix: packaging. now completely based on pyproject.toml (poetry)


3.0.2 (2022-06-14)
__________________
Expand Down
18 changes: 0 additions & 18 deletions MANIFEST.in

This file was deleted.

1 change: 0 additions & 1 deletion VERSION

This file was deleted.

44 changes: 43 additions & 1 deletion poetry.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading

0 comments on commit 9d72943

Please sign in to comment.