Skip to content

Use trusted publisher and environment#278

Merged
eccles merged 3 commits into
mainfrom
dev/paul/use-trusted-publisher-to-upload-to-pypi
Nov 29, 2024
Merged

Use trusted publisher and environment#278
eccles merged 3 commits into
mainfrom
dev/paul/use-trusted-publisher-to-upload-to-pypi

Conversation

@eccles
Copy link
Copy Markdown
Contributor

@eccles eccles commented Nov 29, 2024

Setup PyPi for datatrails-python to use a trusted publisher. Delete any API tokens in PyPi for datatrails-python.

In github repo delete PYPI secrets and create an environmant called release that is restricted to the main branch.

Testing: can only be tested by merging and releasing: if this fails it will be refined and remerged and rereleased. Any wheels so uploaded will be marked as alpha and yanked to prevent access by third parties.

@eccles eccles force-pushed the dev/paul/use-trusted-publisher-to-upload-to-pypi branch from 34dedab to 4c1b08d Compare November 29, 2024 11:51
Setup PyPi for datatrails-python to use a trusted publisher. Delete
any API tokens in PyPi for datatrails-python.

In github repo delete PYPI secrets and create an environmant called
release that is restricted to the main branch.
@eccles eccles force-pushed the dev/paul/use-trusted-publisher-to-upload-to-pypi branch from 4c1b08d to cbe9d7d Compare November 29, 2024 11:57
Comment thread .github/workflows/package.yml Outdated
@eccles eccles requested a review from landintrees November 29, 2024 13:08
Comment thread .github/workflows/package.yml Outdated
Comment thread .github/workflows/package.yml Outdated
@eccles eccles requested a review from waarg November 29, 2024 14:05
@eccles eccles changed the title Use trusted publisher and ennvironment Use trusted publisher and environment Nov 29, 2024
@eccles eccles merged commit f626140 into main Nov 29, 2024
@eccles eccles deleted the dev/paul/use-trusted-publisher-to-upload-to-pypi branch November 29, 2024 14:11
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