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

Verrsion bump restore 11.0.0 with CI fix #875

Merged
merged 4 commits into from
Mar 19, 2025

Conversation

d4r1091
Copy link
Member

@d4r1091 d4r1091 commented Mar 18, 2025

Context

I'm restoring the version Bump here and applying the Fix that triggers the CI to check the Common.xcconfig path correctly.
I'm also adding the check on our CI to be able to trigger a release out of our main-[old-firefox-vanilla-ref] branch.

@d4r1091 d4r1091 requested a review from a team March 18, 2025 16:50
Copy link

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ Recommended focus areas for review

Config Update

Validate that the updated branch filters and file path mapping are intentional and correctly trigger the deployment for release versions.

      - main
      - /^main-\d+$/
name: Deploy release version over Testflight if MARKETING_VERSION file updated
config-path: .circleci/deploy-release.yml
mapping: |
  firefox-ios/Client/Configuration/Common.xcconfig deploy-release true
Version Update

Confirm that bumping the version to 11.0.0 is appropriate and consistent with release requirements.

MARKETING_VERSION = 11.0.0

Copy link

PR Code Suggestions ✨

No code suggestions found for the PR.

Copy link
Collaborator

@lucaschifino lucaschifino left a comment

Choose a reason for hiding this comment

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

Nice 🤞

@d4r1091 d4r1091 merged commit 89fb2fb into main Mar 19, 2025
3 of 4 checks passed
@d4r1091 d4r1091 deleted the dc-version-bump-restore-11.0.0-with-fix branch March 19, 2025 08:38
d4r1091 added a commit that referenced this pull request Mar 19, 2025
d4r1091 added a commit that referenced this pull request Mar 19, 2025
d4r1091 added a commit that referenced this pull request Mar 19, 2025
d4r1091 added a commit that referenced this pull request Mar 19, 2025
d4r1091 added a commit that referenced this pull request Mar 19, 2025
And addition added later via force push to fix the project path + Common.xcconfig path
d4r1091 added a commit that referenced this pull request Mar 19, 2025
And addition added later via force push to fix the project path + Common.xcconfig path as well as some build settings fixes.
d4r1091 added a commit that referenced this pull request Mar 19, 2025
And addition added later via force push to fix the project path + Common.xcconfig path as well as some build settings fixes.
d4r1091 added a commit that referenced this pull request Mar 19, 2025
And addition added later via force push to fix the project path + Common.xcconfig path as well as some build settings fixes.
d4r1091 added a commit that referenced this pull request Mar 20, 2025
And addition added later via force push to fix the project path + Common.xcconfig path as well as some build settings fixes. Also referencing the empty Production file to make it editable.
d4r1091 pushed a commit that referenced this pull request Mar 22, 2025
Some additions are added on top of the referenced PR in the commit.
This is a force-pushed commit to keep the latest version bump to 11.0.0
------
Added the possibility to strictly reference to `main` and `main-xxx` branches to spin up builds.
Major fixes are inherent to the `firefox-ios` path added to the `check_marketing_version.sh` script as well as some `yml` CI files for both GitHub Actions and CircleCI.
We are also referencing an empty Production file (git ignored) to make easy for the CI script to inject the required production-only configs.
d4r1091 pushed a commit that referenced this pull request Mar 22, 2025
Some additions are added on top of the referenced PR in the commit.
This is a force-pushed commit to keep the latest version bump to 11.0.0
------
Added the possibility to strictly reference to `main` and `main-xxx` branches to spin up builds.
Major fixes are inherent to the `firefox-ios` path added to the `check_marketing_version.sh` script as well as some `yml` CI files for both GitHub Actions and CircleCI.
We are also referencing an empty Production file (git ignored) to make easy for the CI script to inject the required production-only configs.
d4r1091 added a commit that referenced this pull request Mar 24, 2025
And addition added later via force push to fix the project path + Common.xcconfig path as well as some build settings fixes. Also referencing the empty Production file to make it editable.
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.

2 participants