Skip to content

Pypi release#2

Merged
roznawsk merged 13 commits intomainfrom
pypi-release
Sep 13, 2023
Merged

Pypi release#2
roznawsk merged 13 commits intomainfrom
pypi-release

Conversation

@roznawsk
Copy link
Copy Markdown
Member

@roznawsk roznawsk commented Sep 11, 2023

Prepare for package release on PyPI:

  • create pyproject.toml configuration
  • change generated code to subpackage of jellyfish
  • fixed issue with docker tests

for testing purposes the package is released on test.pypi: https://test.pypi.org/project/jellyfish-server-sdk/

I think we can fully automatise the release procedure when releasing JF 0.2.0.

@roznawsk roznawsk reopened this Sep 12, 2023
@roznawsk roznawsk requested review from LVala and sgfn September 12, 2023 13:21
@roznawsk roznawsk marked this pull request as ready for review September 12, 2023 13:21
Comment thread generate_client.sh
Comment on lines 6 to +14
openapi-generator-cli generate \
-i https://raw.githubusercontent.com/jellyfish-dev/jellyfish/main/openapi.yaml \
-g python \
-o openapi \
-o $TMP_DIR \
-t templates \
--package-name openapi_client \
--global-property apis,models,modelTests=false,supportingFiles

rm -rf openapi/{docs,test,.github,.openapi-generator,.gitignore,.gitlab-ci.yml,.travis.yml,.openapi-generator-ignore,git_push.sh,README.md,setup.cfg,test-requirements.txt,tox.ini}
--package-name jellyfish.$PACKAGE_NAME \
--global-property apis,models,modelTests=false,apiTests=false,modelDocs=false,apiDocs=false,supportingFiles

rm -rf jellyfish/$PACKAGE_NAME
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This isn't a big issue since we have version control, but you could consider exiting without deleting the old _openapi_client directory if the generate command failed

@roznawsk roznawsk merged commit ebe6b1d into main Sep 13, 2023
@roznawsk roznawsk deleted the pypi-release branch September 13, 2023 07:51
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.

3 participants