-
Notifications
You must be signed in to change notification settings - Fork 84
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
Spruce release workflows #216
Conversation
…date makefile to support uploading spruce to test pypi
@@ -6,10 +6,10 @@ function bumpVersion(currentVersion, bumpType, prerelease) { | |||
if (prerelease) { | |||
newVersion = `${newVersion}.${prerelease}`; | |||
} | |||
core.setOutput('previous_version', currentVersion) | |||
core.setOutput('previous_version_tag', `v${currentVersion}`) | |||
core.setOutput('previous_version', currentVersion); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Made changes in here initially and backed them out, but prettier had some updates.
@@ -23,6 +23,11 @@ package: | |||
|
|||
upload: | |||
poetry publish --verbose --username ${PYPI_USERNAME} --password ${PYPI_PASSWORD} | |||
|
|||
upload-spruce: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
poetry requires a configuration command to set up the repository for test-pypi: https://python-poetry.org/docs/repositories/#publishable-repositories
testing: | ||
uses: './.github/workflows/testing.yaml' | ||
|
||
version-and-release-spruce: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Initially I wanted to leverage publish-to-pypi
which the regular release
workflow calls: https://github.com/pinecone-io/pinecone-python-client/blob/main/.github/workflows/publish-to-pypi.yaml
However, since we're not actually bumping the version for Spruce, it felt odd to make a lot of changes to support that. Instead I opted to use the same flow for publishing that we leverage in nightly-release
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks sensible to me. Nice job.
Problem
We need a way to ship builds of the
spruce
branch.Solution
Modify existing workflows for
nightly-release
andrelease
for shipping artifacts to Test PyPI.Workflows update
pyproject.toml
name
field:pinecone-client-spruce-dev
andpinecone-client-spruce
:pinecone-client-spruce
releases are for use in 3rd party testingpinecone-client-spruce-dev
releases are for use in internal testingType of Change
Test Plan
Trigger and validate workflows either locally or from the repo directly.