Skip to content
This repository has been archived by the owner on Apr 26, 2023. It is now read-only.

dont use pointer in loader #164

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from
Draft

dont use pointer in loader #164

wants to merge 6 commits into from

Conversation

Haaroon
Copy link
Collaborator

@Haaroon Haaroon commented Apr 20, 2023

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change? If yes is this documented?

How was this patch tested?

Are there any further changes required?

@codecov
Copy link

codecov bot commented Apr 20, 2023

Codecov Report

Merging #164 (df08664) into main (9d8db8d) will increase coverage by 0.74%.
The diff coverage is 97.43%.

❗ Current head df08664 differs from pull request most recent head 902eb50. Consider uploading reports for the commit 902eb50 to get more accurate results

@@            Coverage Diff             @@
##             main     #164      +/-   ##
==========================================
+ Coverage   70.13%   70.87%   +0.74%     
==========================================
  Files          68       67       -1     
  Lines       10517    10538      +21     
==========================================
+ Hits         7376     7469      +93     
+ Misses       3141     3069      -72     
Impacted Files Coverage Δ
docbrown/src/core/mod.rs 95.23% <ø> (+52.38%) ⬆️
python/src/graph_loader.rs 0.00% <ø> (ø)
docbrown/src/graph_loader/reddit_hyperlinks.rs 92.18% <93.33%> (+92.18%) ⬆️
docbrown/src/db/graph.rs 89.52% <100.00%> (+0.18%) ⬆️
docbrown/src/graph_loader/lotr_graph.rs 77.77% <100.00%> (ø)
docbrown/src/graph_loader/mod.rs 95.00% <100.00%> (+1.25%) ⬆️

... and 1 file with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant