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

improvement(sync): use native Mutagen daemon by default #6227

Merged
merged 2 commits into from
Jun 28, 2024

Conversation

vvagaytsev
Copy link
Collaborator

What this PR does / why we need it:

This PR flips the default value of GARDEN_ENABLE_NEW_SYNC feature flag to true.
It means that the sync mode will use the native Mutagen instead of the Garden-fork as a synchronization machinery.

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

@vvagaytsev vvagaytsev changed the title feat(sync): use native Mutagen daemon by default improvement(sync): use native Mutagen daemon by default Jun 25, 2024
@vvagaytsev vvagaytsev force-pushed the tool/switch-to-native-mutagen branch from 45aebf9 to 554878d Compare June 25, 2024 15:23
@vvagaytsev vvagaytsev force-pushed the tool/switch-to-native-mutagen branch from 554878d to fed1628 Compare June 27, 2024 09:22
@vvagaytsev vvagaytsev requested a review from twelvemo June 27, 2024 11:22
@vvagaytsev vvagaytsev marked this pull request as ready for review June 27, 2024 11:22
Copy link
Collaborator

@twelvemo twelvemo left a comment

Choose a reason for hiding this comment

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

Some test failures but otherwise looks good!

@vvagaytsev vvagaytsev enabled auto-merge June 27, 2024 14:49
@vvagaytsev vvagaytsev added this pull request to the merge queue Jun 27, 2024
github-merge-queue bot pushed a commit that referenced this pull request Jun 27, 2024
* feat(sync): enable native Mutagen by default

* improvement(sync): update log warnings and hints
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jun 27, 2024
@vvagaytsev vvagaytsev added this pull request to the merge queue Jun 28, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jun 28, 2024
@vvagaytsev vvagaytsev added this pull request to the merge queue Jun 28, 2024
Merged via the queue into main with commit b33d9c8 Jun 28, 2024
40 checks passed
@vvagaytsev vvagaytsev deleted the tool/switch-to-native-mutagen branch June 28, 2024 09:14
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