Skip to content

Conversation

@andrewsy-opal
Copy link
Collaborator

@andrewsy-opal andrewsy-opal commented Nov 27, 2024

Additions:

  1. Update README to point to PyPI install (Commit 1)
  2. Change Python package name to opal_security due to opal being taken (Commit 1)
  3. Remove outdated tests (Commit 2 with autogenerated changes from 1) and move from opal to opal_security
    • Weirdly, we had a models and a model package - seems like the new openapi-codegen uses models, so when I autogenerated the new opal_security package those didn't get ported over, hence they got deleted.
  4. Autogenerate code based on template changes (Commit 2)
  5. Build and publish GitHub action (Commit 3)

@@ -21,8 +21,7 @@
#
# prerequisite: setuptools
# http://pypi.python.org/pypi/setuptools
NAME = "opal"
VERSION = "1.0.0"
Copy link
Contributor

Choose a reason for hiding this comment

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

why are we removing VERSION from setup

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

we're adding scm_version so we use the git tags for the version

@andrewsy-opal andrewsy-opal merged commit bf90052 into main Nov 27, 2024
11 checks passed
@andrewsy-opal andrewsy-opal deleted the andrewsy-pypi branch November 27, 2024 19:43
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