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

Open some API and create public API tests for compatibility #1290

Merged
merged 1 commit into from
Sep 10, 2022

Conversation

lahma
Copy link
Collaborator

@lahma lahma commented Sep 10, 2022

  • change IConstrant to abstract base class for performance
  • have constraints as public sealed types
  • use uniform postfix Constraint for constraints
  • allow changing max statements during runtime
  • remove old TimeConstraint and replace with TimeConstraint2

This should also remove the need of custom fork for RavenDB

@lahma lahma force-pushed the ravendb-compatibility branch from dd93fda to 1aff2f0 Compare September 10, 2022 06:59
* change IConstrant to abstract base class for performance
* have constraints as public sealed types
* use uniform postfix Constraint for constraints
* allow changing max statements during runtime
@lahma lahma force-pushed the ravendb-compatibility branch from 1aff2f0 to 3af05c9 Compare September 10, 2022 07:07
@lahma lahma enabled auto-merge (squash) September 10, 2022 07:08
@lahma lahma merged commit e38849e into sebastienros:main Sep 10, 2022
@lahma lahma deleted the ravendb-compatibility branch September 10, 2022 07:13
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.

1 participant