8000 feat(mj-include): create an include-loader to load from the filesystem by jdrouet · Pull Request #32 · jolimail/mrml-core · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
This repository was archived by the owner on Jul 25, 2023. It is now read-only.

feat(mj-include): create an include-loader to load from the filesystem #32

Merged
merged 2 commits into from
Feb 7, 2023
Merged

feat(mj-include): create an include-loader to load from the filesystem #32

merged 2 commits into from
Feb 7, 2023

Conversation

jdrouet
Copy link
Contributor
@jdrouet jdrouet commented Feb 7, 2023

Signed-off-by: Jérémie Drouet jeremie.drouet@gmail.com

@jdrouet jdrouet force-pushed the include-loader-local branch from 691da82 to eedfcc2 Compare February 7, 2023 14:35
@jdrouet jdrouet marked this pull request as ready for review February 7, 2023 14:40
@jdrouet jdrouet force-pushed the include-loader-local branch from eedfcc2 to 779d077 Compare February 7, 2023 14:44
@codecov
Copy link
codecov bot commented Feb 7, 2023 •
edited
Loading

Codecov Report

Merging #32 (57976d1) into main (cb2cc56) will increase coverage by 0.14%.
The diff coverage is 97.67%.

@@            Coverage Diff             @@
##             main      #32      +/-   ##
==========================================
+ Coverage   94.09%   94.23%   +0.14%     
==========================================
  Files         211      212       +1     
  Lines       10171    10298     +127     
==========================================
+ Hits         9570     9704     +134     
+ Misses        601      594       -7
Impacted Files Coverage Δ
src/prelude/parse/mod.rs 67.56% <ø> (ø)
src/prelude/parse/local_loader.rs 96.42% <96.42%> (ø)
src/prelude/parse/loader.rs 90.78% <100.00%> (+42.30%) ⬆️

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

@jdrouet jdrouet force-pushed the include-loader-local branch 2 times, most recently from 234df04 to 39683cb Compare February 7, 2023 14:58
@jdrouet
feat(mj-include): create an include-loader to load from the filesystem
de3d270 
Signed-off-by: Jérémie Drouet <jeremie.drouet@gmail.com>
@jdrouet jdrouet force-pushed the include-loader-local branch 2 times, most recently from 19c9a35 to 329a57c Compare February 7, 2023 16:13
@jdrouet
test(prelude parse): increase coverage
57976d1 
Signed-off-by: Jérémie Drouet <jeremie.drouet@gmail.com>
@jdrouet jdrouet force-pushed the include-loader-local branch from 329a57c to 57976d1 Compare February 7, 2023 16:17
@jdrouet jdrouet merged commit c83be97 into main Feb 7, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@jdrouet
0