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

Remove access token reference for PyPI #1350

Merged
merged 1 commit into from
Jul 26, 2024
Merged

Conversation

tlento
Copy link
Contributor

@tlento tlento commented Jul 26, 2024

We have changed authentication methods for PyPI to a more secure
approach, so we no longer have to store an access token in GitHub
secrets. This removes the in-code references so we can delete
the secrets themselves.

We have changed authentication methods for PyPI to a more secure
approach, so we no longer have to store an access token in GitHub
secrets. This removes the in-code references so we can delete
the secrets themselves.
@tlento tlento merged commit 19ecaeb into main Jul 26, 2024
43 checks passed
@tlento tlento deleted the remove-cd-pypi-access-token branch July 26, 2024 17:21
tlento added a commit that referenced this pull request Jul 27, 2024
We have changed authentication methods for PyPI to a more secure
approach, so we no longer have to store an access token in GitHub
secrets. This removes the in-code references so we can delete
the secrets themselves.
tlento added a commit that referenced this pull request Jul 30, 2024
We have changed authentication methods for PyPI to a more secure
approach, so we no longer have to store an access token in GitHub
secrets. This removes the in-code references so we can delete
the secrets themselves.
tlento added a commit that referenced this pull request Jul 30, 2024
#1362)

Backport of three PRs required to enable tag-based deployment from the 
`support/dbt-metricflow-0.7.x` branch. Specifically:

- **Enable tag-based deployment for dbt-metricflow (#1349)**
- **Remove access token reference for PyPI (#1350)**
- **Enable Trusted Publishing for PyPI actions (#1356)**
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants