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

Streamline GitHub root handling #1230

Merged
merged 5 commits into from
Sep 15, 2021
Merged

Conversation

bitwiseman
Copy link
Member

@bitwiseman bitwiseman commented Sep 9, 2021

Description

Streamline GitHub root handling.
Making this work required change to GHMyself and how GitHub and GitHubClient get login strings.
I also added tests to cover a number of missing code paths to continue to bring coverage numbers up.

@codecov
Copy link

codecov bot commented Sep 9, 2021

Codecov Report

Merging #1230 (0467f0a) into main (bb72b8d) will increase coverage by 1.44%.
The diff coverage is 77.77%.

Impacted file tree graph

@@             Coverage Diff              @@
##               main    #1230      +/-   ##
============================================
+ Coverage     75.45%   76.90%   +1.44%     
+ Complexity     1910     1885      -25     
============================================
  Files           188      188              
  Lines          6201     5958     -243     
  Branches        369      328      -41     
============================================
- Hits           4679     4582      -97     
+ Misses         1301     1176     -125     
+ Partials        221      200      -21     
Impacted Files Coverage Δ
...ava/org/kohsuke/github/GHAppInstallationToken.java 83.33% <ø> (+30.00%) ⬆️
.../main/java/org/kohsuke/github/GHAuthorization.java 91.66% <ø> (+5.00%) ⬆️
...ava/org/kohsuke/github/GHContentSearchBuilder.java 73.91% <ø> (-2.09%) ⬇️
.../java/org/kohsuke/github/GHContentWithLicense.java 100.00% <ø> (+20.00%) ⬆️
src/main/java/org/kohsuke/github/GHDeployKey.java 0.00% <0.00%> (ø)
src/main/java/org/kohsuke/github/GHInvitation.java 0.00% <0.00%> (ø)
src/main/java/org/kohsuke/github/GHIssueEvent.java 56.52% <ø> (-2.74%) ⬇️
src/main/java/org/kohsuke/github/GHKey.java 100.00% <ø> (+30.00%) ⬆️
.../java/org/kohsuke/github/GHMarketplaceAccount.java 85.71% <ø> (-3.18%) ⬇️
...a/org/kohsuke/github/GHMarketplaceAccountPlan.java 100.00% <ø> (+11.11%) ⬆️
... and 113 more

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 bb72b8d...0467f0a. Read the comment docs.

@bitwiseman bitwiseman force-pushed the task/cleanup-root branch 3 times, most recently from b2bcced to d864757 Compare September 10, 2021 07:18
@bitwiseman bitwiseman force-pushed the task/cleanup-root branch 5 times, most recently from d9cb1bf to d099220 Compare September 14, 2021 17:52
@bitwiseman bitwiseman force-pushed the task/cleanup-root branch 2 times, most recently from 88ef76d to 22cfb00 Compare September 15, 2021 05:53
@bitwiseman bitwiseman marked this pull request as ready for review September 15, 2021 08:16
@bitwiseman bitwiseman merged commit bcb71a3 into hub4j:main Sep 15, 2021
@bitwiseman bitwiseman deleted the task/cleanup-root branch September 15, 2021 08:27
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