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

Sync, blog, part 1 #622

Merged
merged 5 commits into from
Sep 9, 2024
Merged

Sync, blog, part 1 #622

merged 5 commits into from
Sep 9, 2024

Conversation

arogachev
Copy link
Contributor

Q A
Is bugfix?
New feature?
Breaks BC?

@arogachev arogachev marked this pull request as draft September 5, 2024 11:52
@arogachev arogachev self-assigned this Sep 5, 2024
@arogachev arogachev added the status:under development Someone is working on a pull request. label Sep 5, 2024
@arogachev arogachev changed the title Sync Sync, part 1 Sep 8, 2024
@arogachev arogachev requested a review from a team September 8, 2024 09:55
@arogachev arogachev added status:code review The pull request needs review. and removed status:under development Someone is working on a pull request. labels Sep 8, 2024
@arogachev arogachev marked this pull request as ready for review September 8, 2024 09:55
@arogachev arogachev changed the title Sync, part 1 Sync, blog, part 1 Sep 8, 2024
@arogachev arogachev added the type:task Task label Sep 8, 2024
@samdark
Copy link
Member

samdark commented Sep 8, 2024

Tests seems to fail but anyway, this is a step to make it work.

@arogachev
Copy link
Contributor Author

@samdark Yes. I am planning to fix tests in a separate PR.

@arogachev arogachev merged commit 96b9726 into master Sep 9, 2024
4 of 15 checks passed
@arogachev arogachev deleted the sync branch September 9, 2024 09:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status:code review The pull request needs review. type:task Task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants