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

move nucliadb_node/binding to the root [sc-6670] #1154

Merged
merged 1 commit into from
Aug 2, 2023

Conversation

tarekziade
Copy link
Contributor

@tarekziade tarekziade commented Aug 2, 2023

Description

move the nucliadb-node-binding package as a root package

How was this PR tested?

The nucliadb_node GH action and the Makefile were adapted to pick the binding in the right spot.

@tarekziade tarekziade requested a review from a team as a code owner August 2, 2023 09:26
@tarekziade tarekziade marked this pull request as draft August 2, 2023 09:26
@tarekziade tarekziade self-assigned this Aug 2, 2023
@codecov
Copy link

codecov bot commented Aug 2, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (db89866) 85.01% compared to head (bbb2bb2) 85.01%.
Report is 2 commits behind head on main.

❗ Current head bbb2bb2 differs from pull request most recent head 5e62579. Consider uploading reports for the commit 5e62579 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1154   +/-   ##
=======================================
  Coverage   85.01%   85.01%           
=======================================
  Files         402      402           
  Lines       22314    22314           
=======================================
  Hits        18971    18971           
  Misses       3343     3343           
Flag Coverage Δ
ingest 74.50% <ø> (ø)
reader 74.08% <ø> (ø)
utils 85.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@tarekziade tarekziade marked this pull request as ready for review August 2, 2023 09:39
@tarekziade tarekziade force-pushed the tarekziade/refactor-nucliadb-node-binding branch from b57c562 to 82968ec Compare August 2, 2023 13:01
@tarekziade tarekziade changed the title move nucliadb_node/binding to the root move nucliadb_node/binding to the root [sc-6670] Aug 2, 2023
@tarekziade tarekziade force-pushed the tarekziade/refactor-nucliadb-node-binding branch from bbb2bb2 to 5e62579 Compare August 2, 2023 14:07
@tarekziade tarekziade added this pull request to the merge queue Aug 2, 2023
Merged via the queue into main with commit 765557c Aug 2, 2023
35 checks passed
@tarekziade tarekziade deleted the tarekziade/refactor-nucliadb-node-binding branch August 2, 2023 14: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