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

Common sync schema collection for all tests #1399

Conversation

desistefanova
Copy link
Contributor

This PR is created to avoid removing schemas when breaking changes are allowed in dev mode in the new baas version.

@cla-bot cla-bot bot added the cla: yes label Sep 14, 2023
@desistefanova desistefanova changed the base branch from main to ds/core13.19_sync_errors_changes September 14, 2023 14:54
@desistefanova desistefanova changed the title Experimental: Common test schema collection for all tests Experimental: Common sync schema collection for all tests Sep 14, 2023
@desistefanova desistefanova added the no-changelog Used to skip the changelog check label Sep 14, 2023
Copy link
Contributor

@nielsenko nielsenko left a comment

Choose a reason for hiding this comment

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

LGTM

@nielsenko
Copy link
Contributor

.. once all the test pass 😅

@desistefanova desistefanova marked this pull request as ready for review September 15, 2023 09:36
@desistefanova desistefanova changed the title Experimental: Common sync schema collection for all tests Common sync schema collection for all tests Sep 15, 2023
@desistefanova desistefanova merged commit e92848c into ds/core13.19_sync_errors_changes Sep 15, 2023
18 checks passed
@desistefanova desistefanova deleted the ds/common_test_schemas_new__baas branch September 15, 2023 11:19
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 14, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes no-changelog Used to skip the changelog check
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants