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

refactor: awsprovider file restructure #296

Merged
merged 4 commits into from
May 18, 2021

Conversation

eytan-avisror
Copy link
Collaborator

Signed-off-by: Eytan Avisror eytan_avisror@intuit.com

Adds to #294

This separates the package to files by service

Signed-off-by: Eytan Avisror <eytan_avisror@intuit.com>
@eytan-avisror eytan-avisror requested review from a team as code owners May 8, 2021 05:41
@codecov
Copy link

codecov bot commented May 8, 2021

Codecov Report

Merging #296 (85e7189) into master (464de75) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #296   +/-   ##
=======================================
  Coverage   70.05%   70.05%           
=======================================
  Files          19       19           
  Lines        3156     3156           
=======================================
  Hits         2211     2211           
  Misses        804      804           
  Partials      141      141           

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 464de75...85e7189. Read the comment docs.

@eytan-avisror eytan-avisror changed the title refactor awsprovider: file restructure refactor: awsprovider file restructure May 11, 2021
@eytan-avisror eytan-avisror merged commit 63f73b4 into keikoproj:master May 18, 2021
@backjo backjo mentioned this pull request Jun 4, 2021
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.

1 participant