Closed Mastermindaxe closed 3 years ago
@calebcartwright My first try on this :D It seems to be working fine. There's one thing that's kinda bothering me though and not very ergonomic.
Merging #116 into master will not change coverage. The diff coverage is
100.00%
.
@@ Coverage Diff @@
## master #116 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 4 4
Lines 129 141 +12
=========================================
+ Hits 129 141 +12
Impacted Files | Coverage Δ | |
---|---|---|
src/config.rs | 100.00% <100.00%> (ø) |
|
src/hooks.rs | 100.00% <0.00%> (ø) |
|
src/rusty_hook.rs | 100.00% <0.00%> (ø) |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 918bc57...3b17e95. Read the comment docs.
And is there any way for me to see what is not covered by the tests? To fix that code coverage issue. I haven't worked with code coverage tools in Rust yet 😅 This PR is probably more work for you than if you just would have done it yourself 🙈
EDIT: I guess #18 would be helpful. I'm planning to do some more OSS work so I'd be glad to help out. (If I can with my limited knowledge)
And is there any way for me to see what is not covered by the tests? To fix that code coverage issue. I haven't worked with code coverage tools in Rust yet sweat_smile
I think that the Codecov links should be public (https://codecov.io/gh/swellaby/rusty-hook/pull/116/diff?src=pr&el=tree#diff-c3JjL2NvbmZpZy5ycw==). The Coverage is being generated from tarpaulin which is pretty solid, though occasionally imperfect and only available on Linux. If you'd like to run locally and on Mac then I believe the Docker option can work, and on Windows using WSL should work too.
This PR is probably more work for you than if you just would have done it yourself see_no_evil
Not at all! I'd much rather have others contribute and be able to just review than have to do it myself :smile:
EDIT: I guess #18 would be helpful.
Lol exactly!
I'm planning to do some more OSS work so I'd be glad to help out. (If I can with my limited knowledge)
That's great to hear! It's good timing as well with Hacktoberfest fast approaching. There's plenty of work to do here if you're interested and have time
That's great to hear! It's good timing as well with Hacktoberfest fast approaching. There's plenty of work to do here if you're interested and have time
Hell yeah! I have the feeling that this is a small enough repo to get me started to OSS. This is my first PR with code in it ever and I'm very happy how easy it was to work with this repo
Not at all! I'd much rather have others contribute and be able to just review than have to do it myself 😄
Glad to hear that!
Thanks @Mastermindaxe! This is good to go, though would you mind squashing up the commits a bit? Would prefer avoiding adding1db457b2f8c0ce3ab8d45417d308dacd4f667338 and 2d67771783d2782496f63b0bf8ad69ce17bcd15a to the history. Alternatively I can squash while merging
I've never squashed commits so feel free to do that while merging 😅 I'm gonna look into it though so I can do it next time! Thanks!
I've never squashed commits so feel free to do that while merging sweat_smile I'm gonna look into it though so I can do it next time! Thanks!
No worries, I'll take care of it later today. You may find these helpful for future reference:
https://thoughtbot.com/blog/git-interactive-rebase-squash-amend-rewriting-history http://gitready.com/advanced/2009/02/10/squashing-commits-with-rebase.html
Changes
Related Issues