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 special thanks for our first external contributor #242

Merged
merged 2 commits into from
Jun 23, 2023
Merged

Conversation

shadycuz
Copy link
Member

Also fixed an issue with codecov killing PR's because of a really strict codecov check. Also the last release didn't go out correctly, so this will fix that as well.

shadycuz added 2 commits June 23, 2023 18:06
He was the first contributor and he finished the last couple of features
to make unit testing with cloud-radar match the capabilities of the AWS
Cloudformation service.
This check is based on how much coverage a PR has. The current value
was 3%, which meant that 97% of an PR had to be covered for the check to
pass. This was too high and often would stop PR's that were opened by
renovate that only changed dependnency versions and didn't modify and
project code. Changing this to 70% seems like a much more reasonable
target to hit.
@shadycuz shadycuz added minor A new feature has been added. chore labels Jun 23, 2023
@codecov
Copy link

codecov bot commented Jun 23, 2023

Codecov Report

Merging #242 (dd29995) into develop (bd79459) will decrease coverage by 0.63%.
The diff coverage is 90.00%.

@@             Coverage Diff             @@
##           develop     #242      +/-   ##
===========================================
- Coverage    97.02%   96.39%   -0.63%     
===========================================
  Files           10       10              
  Lines          571      638      +67     
  Branches       107      132      +25     
===========================================
+ Hits           554      615      +61     
- Misses          11       14       +3     
- Partials         6        9       +3     
Flag Coverage Δ
unit 96.39% <90.00%> (-0.63%) ⬇️

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

Impacted Files Coverage Δ
src/cloud_radar/cf/unit/_template.py 95.67% <90.00%> (-2.20%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@shadycuz shadycuz merged commit fd4a199 into develop Jun 23, 2023
@shadycuz shadycuz deleted the add_thanks branch June 23, 2023 18:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore minor A new feature has been added.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant