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

Github Actions for Test and Deploy / Update c7n and c7n_mailer versions #65

Merged
merged 31 commits into from
Jan 24, 2022

Conversation

jleopold28
Copy link
Contributor

@jleopold28 jleopold28 commented Jan 20, 2022

Description

  • Adding github actions workflow to replace TravisCI
  • Bump c7n to release 0.9.14
  • Bump c7n-mailer to release 0.6.13
  • Updated dependencies for c7n and c7n-mailer

Testing Done

Tests in Github Actions

Important Notes

TravisCI has been replaced with Github Actions for Test and Release

Contributor License Agreement

Required for external contributors.

By submitting this work for inclusion in manheim-c7n-tools, I agree to the following terms:

  • The contribution included in this request (and any subsequent revisions or versions of it)
    is being made under the same license as the manheim-c7n-tools project (Apache v2).
  • My contribution may perpetually be included in and distributed with manheim-c7n-tools; submitting
    this pull request grants a perpetual, global, unlimited license for it to be used and distributed
    under the terms of manheim-c7n-tools's license.
  • I have the legal power and rights to agree to these terms.

@codecov-commenter
Copy link

codecov-commenter commented Jan 20, 2022

Codecov Report

Merging #65 (35ac1fd) into master (77a904e) will increase coverage by 0.02%.
The diff coverage is 100.00%.

❗ Current head 35ac1fd differs from pull request most recent head a01e6e0. Consider uploading reports for the commit a01e6e0 to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##           master      #65      +/-   ##
==========================================
+ Coverage   61.47%   61.49%   +0.02%     
==========================================
  Files           9        9              
  Lines        1469     1470       +1     
  Branches      288      306      +18     
==========================================
+ Hits          903      904       +1     
  Misses        565      565              
  Partials        1        1              
Impacted Files Coverage Δ
manheim_c7n_tools/notifyonly.py 100.00% <100.00%> (ø)
manheim_c7n_tools/version.py 100.00% <100.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 77a904e...a01e6e0. Read the comment docs.

@jleopold28 jleopold28 changed the title [WIP] Initial Github Actions for Test and Deploy [WIP]Github Actions for Test and Deploy / Update c7n and c7n_mailer versions Jan 20, 2022
@jleopold28 jleopold28 changed the title [WIP]Github Actions for Test and Deploy / Update c7n and c7n_mailer versions Github Actions for Test and Deploy / Update c7n and c7n_mailer versions Jan 21, 2022
jantman
jantman previously approved these changes Jan 24, 2022
Copy link
Contributor

@jantman jantman left a comment

Choose a reason for hiding this comment

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

Looks good, thanks!

jantman
jantman previously approved these changes Jan 24, 2022
jantman
jantman previously approved these changes Jan 24, 2022
@jleopold28 jleopold28 merged commit 0061781 into master Jan 24, 2022
@jleopold28 jleopold28 deleted the github_actions branch January 24, 2022 15:49
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.

3 participants