Skip to content

Use new fluree.db top level api namespace#70

Merged
zonotope merged 16 commits intomainfrom
refactor/db-api-ns
Jul 16, 2024
Merged

Use new fluree.db top level api namespace#70
zonotope merged 16 commits intomainfrom
refactor/db-api-ns

Conversation

@zonotope
Copy link
Contributor

No description provided.

@zonotope zonotope requested a review from a team June 24, 2024 18:14
Copy link
Contributor

@bplatz bplatz left a comment

Choose a reason for hiding this comment

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

@zonotope Is it ok to re-target #73 to main instead of here and merge? That PR also fixes this cljfmt failing test.

Otherwise we can merge this despite the failing tests, and then merge #73.

@zonotope
Copy link
Contributor Author

@zonotope Is it ok to re-target #73 to main instead of here and merge? That PR also fixes this cljfmt failing test.

Otherwise we can merge this despite the failing tests, and then merge #73.

I think the best thing is to merge #73 into this branch, wait for ci to pass, and then merge this branch into main to catch any merge issues (not likely, but possible).

Since #73 is approved, I'll just do that now.

@zonotope
Copy link
Contributor Author

Looks like we have an intermittently failing history test. We can address that separately, so I'm merging now since everything is green.

@zonotope zonotope merged commit 1bf5d62 into main Jul 16, 2024
@zonotope zonotope deleted the refactor/db-api-ns branch July 16, 2024 20:08
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.

2 participants