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

Release v0.9.0 #243

Merged
merged 9 commits into from
Jun 23, 2023
Merged

Release v0.9.0 #243

merged 9 commits into from
Jun 23, 2023

Conversation

shadycuz
Copy link
Member

No description provided.

renovate bot and others added 7 commits June 16, 2023 03:12
I did not mean to refactor the entire function. I just didn't realize
how many edge cases there were when handling dynamic references. I used
chatGPT for the regex. It took many tries before it gave me a working
one. When I asked it why it took so long for us to derive the correct
answer, it replied with a very generic list of reasons of why it might
have taken so long. The top of that list was miscommunication. I asked
it if it thought we miscommunicated and surprisingly it was able to
clearly articulate how we miscommunicated. I am both excited and afraid
of what the next 10 years will bring.
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.
@codecov
Copy link

codecov bot commented Jun 23, 2023

Codecov Report

Merging #243 (57641b1) into master (d1626ad) will decrease coverage by 0.57%.
The diff coverage is 77.41%.

@@            Coverage Diff             @@
##           master     #243      +/-   ##
==========================================
- Coverage   96.95%   96.39%   -0.57%     
==========================================
  Files          10       10              
  Lines         624      638      +14     
  Branches      128      132       +4     
==========================================
+ Hits          605      615      +10     
- Misses         12       14       +2     
- Partials        7        9       +2     
Flag Coverage Δ
unit 96.39% <77.41%> (-0.57%) ⬇️

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% <77.41%> (-1.75%) ⬇️

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

@shadycuz shadycuz changed the title Release X Release v0.9.0 Jun 23, 2023
@shadycuz shadycuz merged commit 7cc6b5a into master Jun 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants