Skip to content

One Repo To Rule Them All #4058

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

Merged
merged 3,018 commits into from
May 8, 2025
Merged

One Repo To Rule Them All #4058

merged 3,018 commits into from
May 8, 2025

Conversation

ricab
Copy link
Collaborator

@ricab ricab commented Apr 24, 2025

No description provided.

Sploder12 and others added 30 commits November 27, 2024 09:26
Fix symlink resolution when mounting
[windows] filter out "ci" from msi package version string
Fix conflict by moving the check for the VirtualBox backend to compile
time too.
Private side of clone of #3740: "Fix multipass_tests target on Ubuntu ARM64 (#3739)"
Update lukka/get-cmake action to v3.31.0
[ci] macos-12 is deprecated, so update to 13 for combining packages
[ci] try to get windows adk with winget
ricab added 5 commits May 5, 2025 16:17
Update LICENCE.TXT and LICENCE.rtf. The latter gets incorporated in the
MSI package. It was regenerated with:

```
pandoc --template=template.rtf -s LICENCE.txt -o LICENCE.rtf
```
[cmake] Abbreviate hash with 8 chars for version
Update licenses in Windows and macOS packages
@ricab ricab marked this pull request as ready for review May 5, 2025 20:04
@ricab
Copy link
Collaborator Author

ricab commented May 5, 2025

Alright, this is ready for review. A few points:

  • The expected macOS and Windows checks are not relevant here: they correspond to the old triggers onto the private repo. New macOS/Windows jobs are run directly here.
  • The CLA check is complaining on commits that were already on full. It doesn't make sense to attempt editing commits from years ago which were already accepted IMO. I have fixed the authorship of any new commits (unless something escaped me).
  • Coverage in full wasn't tracked and it's no surprise that it was lower. A total -0.06% change isn't bad overall, I think.

@georgeliao georgeliao self-requested a review May 6, 2025 13:03
@georgeliao
Copy link
Contributor

@ricab
I have browsed through all the diff between public/merger and private/full, things look good to me. I also did smoke test on all three platforms, everything works fine.

georgeliao
georgeliao previously approved these changes May 7, 2025
@ricab
Copy link
Collaborator Author

ricab commented May 7, 2025

Thanks @georgeliao.

How about you @sharder996? Does this have your stamp of approval?

@sharder996
Copy link
Collaborator

@ricab Yup, +1 from me!

sharder996
sharder996 previously approved these changes May 7, 2025
@ricab
Copy link
Collaborator Author

ricab commented May 7, 2025

Muhahaha

Sauron

@ricab ricab enabled auto-merge May 7, 2025 19:10
@ricab ricab dismissed stale reviews from sharder996 and georgeliao via b49e021 May 8, 2025 12:52
@ricab
Copy link
Collaborator Author

ricab commented May 8, 2025

Alright, this isn't going to merge because it keeps waiting for the old checks. I have dropped the commit to skip doc checks, since that is now in through #4076. @sharder996 or @georgeliao, could one of you please give this a green checkmark? Then I'll merge manually.

@ricab ricab disabled auto-merge May 8, 2025 12:59
@ricab ricab merged commit b086fb7 into main May 8, 2025
14 of 15 checks passed
@ricab ricab deleted the merger branch May 8, 2025 13:00
@ricab
Copy link
Collaborator Author

ricab commented May 8, 2025

dance

@townsend2010
Copy link
Contributor

Just noticed this was merged. What a miraculous day!!!!

@ricab ricab added this to the 1.16.0 milestone Jun 13, 2025
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.

8 participants