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

Add DB v6 feature flag #2288

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Add DB v6 feature flag #2288

wants to merge 1 commit into from

Conversation

wagoodman
Copy link
Contributor

@wagoodman wagoodman commented Nov 27, 2024

Adds the GRYPE_EXP_DBV6 feature flag while we are developing V6 features and wires up the db * commands to respond to this. The top-level API has not been wired up to respond to this yet, as that will require more breaking changes as well as coordination with the matchers/search work in progress. I've tried to cover new functions in the commands package with tests, but did not add tests for existing functions (most of which did not have tests).

DB commands summary:

  • db check: responds to v6, new tests added
  • db delete: responds to v6, tests in curator already
  • db diff: not modified
  • db import: responds to v6, tests in curator already
  • db list: responds to v6, new tests added
  • db providers: responds to v6, new tests added
  • db search: partially implemented v6 path
  • db status: responds to v6, new tests added
  • db update: responds to v6, tests in curator already

@wagoodman wagoodman added the changelog-ignore Don't include this issue in the release changelog label Nov 27, 2024
@wagoodman wagoodman self-assigned this Nov 27, 2024
@wagoodman wagoodman force-pushed the add-v6-feature-flag branch 2 times, most recently from f3464b5 to 8c5c2ac Compare November 27, 2024 21:45
@wagoodman wagoodman changed the title Add db v6 feature flag Add DB v6 feature flag Nov 27, 2024
@wagoodman wagoodman requested a review from a team November 27, 2024 21:51
@wagoodman wagoodman marked this pull request as ready for review November 27, 2024 21:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog-ignore Don't include this issue in the release changelog
Projects
Status: In Review
Development

Successfully merging this pull request may close these issues.

1 participant