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

Use the shared GitHub Actions workflows for buildpack releases #58

Merged
merged 12 commits into from
Aug 2, 2023

Conversation

colincasey
Copy link
Contributor

@colincasey colincasey commented Jun 28, 2023

This sets up the release automation scripts from languages-github-actions for the Python buildpack.

Also adds the target Docker repository to buildpack.toml to align with the metadata used by other Heroku buildpacks.

GUS-W-13658750.

This sets up the release automation scripts from [languages-github-actions](https://github.com/heroku/languages-github-actions) for the Python buildpack.

Also adds the target Docker repository to `buildpack.toml` to align with the metadata used by other Heroku buildpacks.

[W-13658750](https://gus.lightning.force.com/lightning/r/ADM_Work__c/a07EE00001Ul35kYAB/view)
@colincasey colincasey requested a review from a team as a code owner June 28, 2023 14:58
@edmorley edmorley added the skip changelog Skip the check-changelog check label Jun 28, 2023
.github/workflows/prepare-release.yml Outdated Show resolved Hide resolved
.github/workflows/release.yml Outdated Show resolved Hide resolved
.github/workflows/prepare-release.yml Outdated Show resolved Hide resolved
.github/workflows/release.yml Outdated Show resolved Hide resolved
.github/workflows/release.yml Show resolved Hide resolved
buildpack.toml Outdated Show resolved Hide resolved
.github/workflows/release.yml Outdated Show resolved Hide resolved
@edmorley
Copy link
Member

Thank you for working on this btw! I'll have another review pass once I've had a chance to test some more, and once we've triaged some more of https://github.com/heroku/languages-github-actions/issues :-)

Copy link
Member

@edmorley edmorley left a comment

Choose a reason for hiding this comment

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

Marking changes requested to make the current state clearer.

This is blocked on:

...but I'm very much looking forward to this landing! :-)

@colincasey colincasey requested a review from edmorley August 1, 2023 15:01
@edmorley edmorley changed the title Use the shared release automation for Python buildpack Use the shared GitHub Actions workflows for buildpack releases Aug 2, 2023
Copy link
Member

@edmorley edmorley left a comment

Choose a reason for hiding this comment

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

Amazing, thank you! 😄

buildpack.toml Outdated Show resolved Hide resolved
.github/workflows/release.yml Outdated Show resolved Hide resolved
.github/workflows/prepare-release.yml Outdated Show resolved Hide resolved
.github/workflows/prepare-release.yml Outdated Show resolved Hide resolved
@edmorley edmorley merged commit de38a47 into main Aug 2, 2023
@edmorley edmorley deleted the release_automation branch August 2, 2023 19:16
colincasey added a commit to heroku/languages-github-actions that referenced this pull request Aug 3, 2023
colincasey added a commit to heroku/languages-github-actions that referenced this pull request Aug 3, 2023
We can remove this fallback key for reading the repository to publish to now that the following PRs have been merged:

- heroku/buildpacks-go#124
- heroku/buildpacks-php#15
- heroku/buildpacks-ruby#184
- heroku/buildpacks-jvm#555
- heroku/buildpacks-nodejs#610
- heroku/buildpacks-python#58
- heroku/buildpacks-procfile#156
colincasey added a commit to heroku/languages-github-actions that referenced this pull request Aug 3, 2023
We can remove this fallback key for reading the repository to publish to now that the following PRs have been merged:

- heroku/buildpacks-go#124
- heroku/buildpacks-php#15
- heroku/buildpacks-ruby#184
- heroku/buildpacks-jvm#555
- heroku/buildpacks-nodejs#610
- heroku/buildpacks-python#58
- heroku/buildpacks-procfile#156
colincasey added a commit to heroku/languages-github-actions that referenced this pull request Aug 4, 2023
* Post PR #91 Cleanup

We can remove these separated workflows now that the following PRs have been merged:

- heroku/buildpacks-jvm#554
- heroku/buildpacks-php#14
- heroku/buildpacks-go#123
- heroku/buildpacks-ruby#183
- heroku/buildpacks-nodejs#609
- heroku/buildpacks-python#58
- heroku/buildpacks-procfile#156

* PR feedback
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip changelog Skip the check-changelog check
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants