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

Github Actions revamp: tickets/DM-42231C #135

Merged
merged 2 commits into from
Jan 19, 2024
Merged

Github Actions revamp: tickets/DM-42231C #135

merged 2 commits into from
Jan 19, 2024

Conversation

athornton
Copy link
Collaborator

@athornton athornton commented Jan 15, 2024

Should be rebased once #130 is merged. Replacement for #134 that doesn't cross forks, to see if that gets the ghcr.io push working.

To get PyPi uploads working, Datopian will need to set up a PyPi Trusted Publisher as described in https://docs.pypi.org/trusted-publishers/ ; the environment name should be pypi.

@athornton
Copy link
Collaborator Author

In order to make artifacts at ghcr.io useful, someone with admin privileges on the GitHub Datopian Org will need to allow these packages to be public: cf. https://github.com/orgs/community/discussions/26014

@rufuspollock
Copy link
Member

#130 is merged ...

PS: sorry for slow responses this week - again offline quite a bit. feel free to ping @anuveyatsu to ask him to merge!

@athornton
Copy link
Collaborator Author

Rebased and ready for review by @anuveyatsu .

Copy link
Member

@demenech demenech left a comment

Choose a reason for hiding this comment

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

LGTM. Merging this and will check with admins about how we can make the artifacts public.

CC: @anuveyatsu

@demenech demenech merged commit fd3925c into main Jan 19, 2024
5 checks passed
@demenech demenech deleted the tickets/DM-42231C branch January 19, 2024 12:47
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