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

release: publish emojivoto as yml instead of zip #640

Merged
merged 1 commit into from
Jun 27, 2024

Conversation

davidweisse
Copy link
Contributor

This changes the released Emojivoto deployment from a zip file to a single YAML file.

@davidweisse davidweisse requested a review from burgerdev June 26, 2024 13:52
@davidweisse davidweisse requested a review from katexochen as a code owner June 26, 2024 13:52
@davidweisse davidweisse added the no changelog PRs not listed in the release notes label Jun 26, 2024
Copy link

github-actions bot commented Jun 26, 2024

PR Preview Action v1.4.7
Preview removed because the pull request was closed.
2024-06-27 12:12 UTC

@davidweisse davidweisse force-pushed the dav/emojivoto-zip branch 2 times, most recently from 6adb926 to 5bd1aab Compare June 26, 2024 14:20
Copy link
Contributor

@burgerdev burgerdev left a comment

Choose a reason for hiding this comment

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

The change lgtm, but I wonder whether we should just generate all the released yaml with the CLI in the first place. cc @katexochen

@katexochen
Copy link
Member

The change lgtm, but I wonder whether we should just generate all the released yaml with the CLI in the first place. cc @katexochen

We have tickets for that, but iirc we decided to postpone implementation.

@davidweisse davidweisse force-pushed the dav/emojivoto-zip branch 2 times, most recently from 81b5b8f to 60ca136 Compare June 27, 2024 11:38
@davidweisse davidweisse merged commit cfe0de1 into main Jun 27, 2024
9 checks passed
@davidweisse davidweisse deleted the dav/emojivoto-zip branch June 27, 2024 12:11
katexochen added a commit that referenced this pull request Jul 10, 2024
This was re-introduced by mistake when rebasing #652. Originally removed in #640

Signed-off-by: Paul Meyer <[email protected]>
katexochen added a commit that referenced this pull request Jul 10, 2024
This was re-introduced by mistake when rebasing #652. Originally removed in #640

Signed-off-by: Paul Meyer <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no changelog PRs not listed in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants