Skip to content

pypackage#3

Open
DragosDumitrache wants to merge 5 commits intomearlboro:mainfrom
DragosDumitrache:pypackage
Open

pypackage#3
DragosDumitrache wants to merge 5 commits intomearlboro:mainfrom
DragosDumitrache:pypackage

Conversation

@DragosDumitrache
Copy link

@DragosDumitrache DragosDumitrache commented Jul 22, 2023

This does a few things:

  1. Adds a GitHub action workflow for running tests and publishing if on the main branch and tests pass
  2. Organises this as a package, with top-level inheriting the different CLI groups
  3. Tests are grouped as pytest tests. Currently failing, my bad, I realised you probably overrode variables, will rework those out

Tests are now working, you can check by running pytest in your local venv. 2 of them are failing, but seems like a legit failure

I didn't port over the plot, wasn't quite sure how you wanted to test that

Packaged with poetry instead of pipenv, and in your venv now you'll get a pyflocks CLI, you can see its usage with

pyflocks --help

This CLI should include all other commands/groups you had defined

Comment on lines +23 to +26
# TODO (mis): add this secret to your repository
run: |
poetry config virtualenvs.create false
poetry config pypi-token.pypi ${{ SECRETS.PYPI_TOKEN }}
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You need to set this secret up from PYPI @mearlboro

@mearlboro mearlboro self-assigned this Apr 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants