Skip to content
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

Add example with parametrized test #50

Merged
merged 2 commits into from
Jun 22, 2023
Merged

Add example with parametrized test #50

merged 2 commits into from
Jun 22, 2023

Conversation

navidcy
Copy link
Contributor

@navidcy navidcy commented Jun 22, 2023

No description provided.

@codecov
Copy link

codecov bot commented Jun 22, 2023

Codecov Report

Merging #50 (ab579dd) into main (abf630d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #50   +/-   ##
=======================================
  Coverage   10.72%   10.72%           
=======================================
  Files           2        2           
  Lines         429      429           
=======================================
  Hits           46       46           
  Misses        383      383           

@navidcy navidcy requested review from angus-g and ashjbarnes June 22, 2023 12:09
Copy link
Collaborator

@angus-g angus-g left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine!

@navidcy
Copy link
Contributor Author

navidcy commented Jun 22, 2023

mostly to serve as a placeholder to show us the way

@navidcy navidcy merged commit 663b5f3 into main Jun 22, 2023
@navidcy navidcy deleted the ncc/parametrized-tests branch June 22, 2023 12:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants