Skip to content

refactor: use local-auth for 3 legged samples#2058

Merged
JustinBeckwith merged 2 commits intomasterfrom
localauth
Apr 27, 2020
Merged

refactor: use local-auth for 3 legged samples#2058
JustinBeckwith merged 2 commits intomasterfrom
localauth

Conversation

@JustinBeckwith
Copy link
Contributor

No description provided.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Apr 25, 2020
@codecov
Copy link

codecov bot commented Apr 25, 2020

Codecov Report

Merging #2058 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2058   +/-   ##
=======================================
  Coverage   38.91%   38.91%           
=======================================
  Files           6        6           
  Lines        1074     1074           
  Branches        8        8           
=======================================
  Hits          418      418           
  Misses        656      656           

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 0b8d04c...b48eb5e. Read the comment docs.

Copy link

@bcoe bcoe left a comment

Choose a reason for hiding this comment

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

this is delightful, I think it demonstrates that this is the right refactor that we're deleting 200 lines of code.

@JustinBeckwith JustinBeckwith added automerge Merge the pull request once unit tests and other checks pass. and removed cla: yes This human has signed the Contributor License Agreement. labels Apr 27, 2020
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Apr 27, 2020
@JustinBeckwith JustinBeckwith merged commit bd0accc into master Apr 27, 2020
@JustinBeckwith JustinBeckwith deleted the localauth branch December 13, 2020 18:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge Merge the pull request once unit tests and other checks pass. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants