-
Notifications
You must be signed in to change notification settings - Fork 45
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
[feat] variables context #149
Conversation
- variables are added via "Comtrya.yaml": ```yaml variables: fkbr: "sxoe" kuci: "xhpv" ``` which yields ```bash DEBUG context="variables" key="fkbr" value="sxoe" DEBUG context="variables" key="kuci" value="xhpv" ```
Resolves #10. |
Yeah. I rebased on "upstream" and now everything is broken. I fix it |
- sidequest: fix clippy warnings
@icepuma Oh, sorry about that - I merged some file.link changes that interfere'd with this. |
Already fixed the failing tests and some clippy warnings :) |
assert_eq!(variables_context_values.is_some(), true); | ||
assert_eq!( | ||
variables_context_values.unwrap().get("ship_name").unwrap(), | ||
"Jack O'Neill" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤣
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't lie when I said that I would clutter the codebase with SG-1 references :D
Assuming the tests pass, this is good to go. Great effort, @icepuma |
Codecov Report
@@ Coverage Diff @@
## main #149 +/- ##
==========================================
+ Coverage 48.15% 52.17% +4.02%
==========================================
Files 46 47 +1
Lines 1838 1748 -90
==========================================
+ Hits 885 912 +27
+ Misses 953 836 -117
Continue to review full report at Codecov.
|
variables are added via "Comtrya.yaml":
which yields