Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat!: public release #101

Merged
merged 1 commit into from
Apr 1, 2024
Merged

feat!: public release #101

merged 1 commit into from
Apr 1, 2024

Conversation

moorec-aws
Copy link
Contributor

What was the problem/requirement? (What/Why)

We're about to publicly release this library. We need to prep the build process for that.

What was the solution? (How)

  • Update the build script to no longer define a PIP_INDEX_URL -- the environment variable
    that we used to interface with our internal repository during private development.

  • Update the code quality check to pull deps from the public PyPI, so
    that we're testing as customers would use it.

  • Use the public PyPI in the release flows since all deps are now
    available publicly. This ensures that the artifact we release can be
    built & used by anyone using PyPI.

What is the impact of this change?

Release readiness

How was this change tested?

hatch run lint
hatch run test

Was this change documented?

No

Is this a breaking change?

No, but it is being marked as one to force a minor version bump when we release.


By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@moorec-aws moorec-aws requested a review from a team as a code owner March 30, 2024 02:50
Signed-off-by: Charles Moore <[email protected]>
@moorec-aws moorec-aws merged commit c9be773 into mainline Apr 1, 2024
12 checks passed
@moorec-aws moorec-aws deleted the public-release branch April 1, 2024 17:08
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