Skip to content

Fix trigger all tests branch#4135

Merged
aboedo merged 1 commit into
mainfrom
andy/fix-trigger-all-tests-again
Jul 30, 2024
Merged

Fix trigger all tests branch#4135
aboedo merged 1 commit into
mainfrom
andy/fix-trigger-all-tests-again

Conversation

@aboedo

@aboedo aboedo commented Jul 30, 2024

Copy link
Copy Markdown
Member

Fix trigger all tests branch (again).

This was a pretty obvious bug - I was using the output of the get-branch step, but I never set the id of the step to that.

This setup now more closely matches the solution laid out as a github comment

@aboedo aboedo added the ci label Jul 30, 2024
@aboedo aboedo requested a review from a team July 30, 2024 15:43
@aboedo aboedo self-assigned this Jul 30, 2024

@JayShortway JayShortway left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Niiiceee!

@aboedo aboedo merged commit 781151d into main Jul 30, 2024
@aboedo aboedo deleted the andy/fix-trigger-all-tests-again branch July 30, 2024 16:37
jamesrb1 pushed a commit that referenced this pull request Aug 7, 2024
Fix trigger all tests branch (again). 

This was a pretty obvious bug - I was using the output of the
`get-branch` step, but I never set the id of the step to that.

This setup now more closely matches the [solution laid out as a github
comment](CircleCI-Public/trigger-circleci-pipeline-action#61 (comment))
MarkVillacampa pushed a commit that referenced this pull request Aug 7, 2024
**This is an automatic release.**

### Bugfixes
* Fix Paywalls crash on iOS 18 beta (#4154) via Andy Boedo (@aboedo)
### Dependency Updates
* Bump danger from 9.4.3 to 9.5.0 (#4143) via dependabot[bot]
(@dependabot[bot])
* Bump nokogiri from 1.16.6 to 1.16.7 (#4129) via dependabot[bot]
(@dependabot[bot])
* Bump fastlane from 2.221.1 to 2.222.0 (#4130) via dependabot[bot]
(@dependabot[bot])
### Other Changes
* Update deployment targets for tests (#4145) via Andy Boedo (@aboedo)
* Deploy purchaserTester: clean up dry-run parameter (#4140) via Andy
Boedo (@aboedo)
* Clean up API Testers (#4141) via Andy Boedo (@aboedo)
* More project structure cleanup (#4131) via Andy Boedo (@aboedo)
* temporarily disables purchasetester deploy (#4133) via Andy Boedo
(@aboedo)
* Fix trigger all tests branch (#4135) via Andy Boedo (@aboedo)
* Clean up XCWorkspace and testing apps (#4111) via Andy Boedo (@aboedo)
* tests trigger: add target-branch parameter to trigger from the right
branch (#4121) via Andy Boedo (@aboedo)
* Re-added the RevenueCatUI tests job on every commit (#4113) via Andy
Boedo (@aboedo)
nyeu pushed a commit that referenced this pull request Oct 2, 2024
Fix trigger all tests branch (again). 

This was a pretty obvious bug - I was using the output of the
`get-branch` step, but I never set the id of the step to that.

This setup now more closely matches the [solution laid out as a github
comment](CircleCI-Public/trigger-circleci-pipeline-action#61 (comment))
nyeu pushed a commit that referenced this pull request Oct 2, 2024
**This is an automatic release.**

### Bugfixes
* Fix Paywalls crash on iOS 18 beta (#4154) via Andy Boedo (@aboedo)
### Dependency Updates
* Bump danger from 9.4.3 to 9.5.0 (#4143) via dependabot[bot]
(@dependabot[bot])
* Bump nokogiri from 1.16.6 to 1.16.7 (#4129) via dependabot[bot]
(@dependabot[bot])
* Bump fastlane from 2.221.1 to 2.222.0 (#4130) via dependabot[bot]
(@dependabot[bot])
### Other Changes
* Update deployment targets for tests (#4145) via Andy Boedo (@aboedo)
* Deploy purchaserTester: clean up dry-run parameter (#4140) via Andy
Boedo (@aboedo)
* Clean up API Testers (#4141) via Andy Boedo (@aboedo)
* More project structure cleanup (#4131) via Andy Boedo (@aboedo)
* temporarily disables purchasetester deploy (#4133) via Andy Boedo
(@aboedo)
* Fix trigger all tests branch (#4135) via Andy Boedo (@aboedo)
* Clean up XCWorkspace and testing apps (#4111) via Andy Boedo (@aboedo)
* tests trigger: add target-branch parameter to trigger from the right
branch (#4121) via Andy Boedo (@aboedo)
* Re-added the RevenueCatUI tests job on every commit (#4113) via Andy
Boedo (@aboedo)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants