Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update filter conformance uris #540

Merged
merged 2 commits into from
Mar 8, 2023
Merged

Conversation

gadomski
Copy link
Member

@gadomski gadomski commented Mar 8, 2023

Related Issue(s):

Description:

Not all of these URIs are used, and some of them needed a name change, but I kept things as similar as I could.

PR Checklist:

  • Code is formatted and linted (run pre-commit run --all-files)
  • Tests pass (run make test)
  • Documentation has been updated to reflect changes, if applicable, and docs build successfully (run make docs)
  • Changes are added to the CHANGELOG.

@gadomski gadomski added this to the 2.4.4 milestone Mar 8, 2023
@gadomski gadomski requested a review from philvarner March 8, 2023 16:32
@gadomski gadomski self-assigned this Mar 8, 2023
@gadomski gadomski force-pushed the issues/539-conformance-classes branch from b5e9ca7 to b2433ff Compare March 8, 2023 16:33
Copy link
Collaborator

@philvarner philvarner left a comment

Choose a reason for hiding this comment

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

comments inline

@gadomski gadomski requested a review from philvarner March 8, 2023 19:43
@gadomski gadomski merged commit 3b517e6 into master Mar 8, 2023
@gadomski gadomski deleted the issues/539-conformance-classes branch March 8, 2023 19:53
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Wrong CQL Text/JSON conformance classes
2 participants