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

fix: remove e2e test account credentials #875

Merged
merged 1 commit into from
Jan 9, 2023

Conversation

letsbelopez
Copy link
Contributor

Issue #, if available:

https://app.asana.com/0/1203493556028707/1203510624767048/f

Description of changes:

Removes the e2e test account credentials and uses github oidc and aws account integration instead.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@letsbelopez letsbelopez requested a review from a team as a code owner January 5, 2023 17:36
hein-j
hein-j previously approved these changes Jan 5, 2023
Milan-Shah
Milan-Shah previously approved these changes Jan 5, 2023
@letsbelopez letsbelopez dismissed stale reviews from Milan-Shah and hein-j via f7f2bca January 5, 2023 18:04
@letsbelopez letsbelopez force-pushed the removeTestAccountCredentials branch 6 times, most recently from 092e222 to 03a708d Compare January 6, 2023 01:45
Milan-Shah
Milan-Shah previously approved these changes Jan 6, 2023
@codecov-commenter
Copy link

Codecov Report

Merging #875 (6a0a668) into main (f2311c3) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #875   +/-   ##
=======================================
  Coverage   92.78%   92.78%           
=======================================
  Files         115      115           
  Lines        4853     4853           
  Branches     1404     1404           
=======================================
  Hits         4503     4503           
  Misses        330      330           
  Partials       20       20           

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 f2311c3...6a0a668. Read the comment docs.

@letsbelopez letsbelopez force-pushed the removeTestAccountCredentials branch 2 times, most recently from ea70548 to edce326 Compare January 7, 2023 00:27
Milan-Shah
Milan-Shah previously approved these changes Jan 9, 2023
hein-j
hein-j previously approved these changes Jan 9, 2023
@letsbelopez letsbelopez dismissed stale reviews from hein-j and Milan-Shah via 896ebd9 January 9, 2023 21:04
@letsbelopez letsbelopez force-pushed the removeTestAccountCredentials branch from edce326 to 896ebd9 Compare January 9, 2023 21:04
@letsbelopez letsbelopez merged commit 5dcd69b into main Jan 9, 2023
@letsbelopez letsbelopez deleted the removeTestAccountCredentials branch January 9, 2023 22:18
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.

5 participants