From af035425ed52af1844d0ca321b51cca424c4b4fa Mon Sep 17 00:00:00 2001 From: Arya Massarat <23412689+aryarm@users.noreply.github.com> Date: Tue, 31 Oct 2023 18:46:41 -0700 Subject: [PATCH] also perform command line tests --- .github/workflows/tests.yml | 4 ++++ PUBLISHING.rst | 13 +++---------- 2 files changed, 7 insertions(+), 10 deletions(-) diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml index 681f0d51..3d7d3fb6 100644 --- a/.github/workflows/tests.yml +++ b/.github/workflows/tests.yml @@ -39,6 +39,10 @@ jobs: shell: bash -el {0} run: | python -m pytest --cov=. --cov-report term-missing --cov-fail-under 89 --doctest-modules --junitxml=junit/test-results-${{ matrix.python-version }}.xml + - name: Test command line + shell: bash -el {0) + run: | + ./test/cmdline_tests.sh - name: Upload pytest test results uses: actions/upload-artifact@v3 with: diff --git a/PUBLISHING.rst b/PUBLISHING.rst index 2753e56b..aa2d8f0e 100644 --- a/PUBLISHING.rst +++ b/PUBLISHING.rst @@ -42,7 +42,6 @@ Once changes have been made to develop that are ready to be published, first cho Then go through the steps of merging the changes into the master branch: -#. Run :code:`pytest` and make sure all the tests pass. Then run :code:`./test/cmdline_tests.sh` and make sure those tests pass. #. Change the 'Unreleased Changes' section of :code:`RELEASE_NOTES.rst` to the new version number. #. Check if any changes have been made that have not yet been documented in the release notes. If so, document them. #. Submit a pull request from develop into master on the github webiste. @@ -53,15 +52,9 @@ Then go through the steps of publishing the changed code to PyPI: 1. :code:`cd` into the root of your clone of the trtools repo, checkout master and pull the latest change. 2. Run :code:`rm -rf build dist *.egg-info` to make sure all previous build artifacts are removed -3. Run :code:`python setup.py sdist bdist_wheel` to build the package. - - This will create the warning:: - - UserWarning: Unknown distribution option: 'license_file' warnings.warn(msg) - - You can ignore this warning: the 'license_file' option is necessary for creating the build artifacts - -4. Run :code:`twine upload dist/*` to upload the build to PyPI +3. Run :code:`pip install build` to build the package. +4. Run :code:`python -m build --wheel --sdist` to create the package distribution files. +5. Run :code:`twine upload dist/*` to upload the distribution to PyPI Lastly, the change needs to be published to bioconda.