Skip to content
This repository was archived by the owner on Aug 28, 2025. It is now read-only.

Conversation

@Borda
Copy link
Contributor

@Borda Borda commented Feb 5, 2022

What does this PR do?

Add some testing of internal CLI
Resolves #62

PR review

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.

Did you have fun?

Make sure you had fun coding 🙃

@Borda Borda added the ci/cd label Feb 5, 2022
@Borda Borda force-pushed the ci/update branch 2 times, most recently from a9f2bfc to d0a78c9 Compare February 6, 2022 08:39
@codecov
Copy link

codecov bot commented Feb 6, 2022

Codecov Report

❗ No coverage uploaded for pull request base (main@34aa043). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #140   +/-   ##
=======================================
  Coverage        ?   69.74%           
=======================================
  Files           ?        2           
  Lines           ?      347           
  Branches        ?        0           
=======================================
  Hits            ?      242           
  Misses          ?      105           
  Partials        ?        0           
Flag Coverage Δ
Linux 69.74% <0.00%> (?)
pytest 69.74% <0.00%> (?)

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


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 34aa043...5bc4f65. Read the comment docs.

@Borda Borda marked this pull request as ready for review February 6, 2022 20:29
@Borda Borda requested review from SkafteNicki and rohitgr7 February 6, 2022 21:26
@Borda Borda self-assigned this Feb 6, 2022
Co-authored-by: Aki Nitta <nitta@akihironitta.com>
@Borda Borda enabled auto-merge (squash) February 7, 2022 07:44
@Borda Borda disabled auto-merge February 7, 2022 07:52
@Borda Borda merged commit 400dd2a into main Feb 7, 2022
@Borda Borda deleted the ci/update branch February 7, 2022 07:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add test for HelperCLI class

3 participants