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

[AUTOMATIC] Release/5.2.0 #4077

Merged
merged 3 commits into from
Jul 18, 2024
Merged

[AUTOMATIC] Release/5.2.0 #4077

merged 3 commits into from
Jul 18, 2024

Conversation

RCGitBot
Copy link
Contributor

This is an automatic release.

New Features

Bugfixes

Other Changes

@RCGitBot RCGitBot requested a review from a team July 17, 2024 17:34
The deploy workflow didn't run correctly, because of a regression
introduced in #4025 .

The problem is that we replaced the tag filters with a `when` clause,
but the job itself was relying on us using the filter and running the
second part of the deploy workflow from the tag itself and not the
branch.

I.e.: it wasn't entirely a chain of jobs, rather, there were two
separate chains, one for releases and one for tags, running in the same
job.

This PR splits the jobs into two parts: one that creates the tag, and
one that fires when a new tag is pushed, to create a release for it
@aboedo
Copy link
Member

aboedo commented Jul 17, 2024

well, release-checks should now fail because the tag already exists

@aboedo
Copy link
Member

aboedo commented Jul 17, 2024

unless I beat it to the punch Edit: I did! it should pass

Follow-up to #4079,
removes the `when` and re-introduces filters as the way to trigger the
job since `when` doesn't seem to work for tags
@aboedo
Copy link
Member

aboedo commented Jul 17, 2024

(I cancelled release checks because I pushed the tag manually the second time, and it was before that job could run)

@joshdholtz
Copy link
Member

@aboedo Going to merge this since everything but the docs is shipped 💪

Copy link
Member

@joshdholtz joshdholtz left a comment

Choose a reason for hiding this comment

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

🚢

@joshdholtz joshdholtz merged commit 5c0c924 into main Jul 18, 2024
37 of 39 checks passed
@joshdholtz joshdholtz deleted the release/5.2.0 branch July 18, 2024 00:57
nyeu pushed a commit that referenced this pull request Oct 2, 2024
**This is an automatic release.**

### New Features
* Added new paywall template to support multiple tiered subscriptions
(#4022) via Josh Holtz (@joshdholtz)
### Bugfixes
* Fix certain completion blocks not being dispatched on the main thread
(#4058) via Mark Villacampa (@MarkVillacampa)
* Only checks staged files for leftover API keys. (#4073) via
JayShortway (@JayShortway)
### Other Changes
* [Ci] Fix trigger to run all tests from github actions (#4075) via Andy
Boedo (@aboedo)
* added new workflow to trigger all tests (#4051) via Andy Boedo
(@aboedo)
* Reduce CI jobs (#4025) via Andy Boedo (@aboedo)

---------

Co-authored-by: Andy Boedo <[email protected]>
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.

4 participants