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

feat: support github enterprise api #418

Merged

Conversation

ricardojdsilva87
Copy link
Contributor

Pull Request

Proposed Changes

This Pull request is based on the already merged github/evergreen#256 to give support to GitHub Enterprise login using a GitHub App also created on GitHub Enterprise

Code coverage

---------- coverage: platform darwin, python 3.13.0-final-0 ----------
Name                          Stmts   Miss  Cover   Missing
-----------------------------------------------------------
auth.py                          28      4    86%   50, 80-82
classes.py                       11      0   100%
config.py                        86      0   100%
discussions.py                   15      0   100%
issue_metrics.py                 65     10    85%   92, 110, 124-127, 139, 149, 154, 366
json_writer.py                   64      3    95%   95, 198-199
labels.py                        71      5    93%   55, 78, 94, 114, 118
markdown_helpers.py              11      0   100%
markdown_writer.py              105      5    95%   149, 263, 273, 283, 293
most_active_mentors.py           46     13    72%   97-113, 116-130
search.py                        66     23    65%   46-56, 81-105, 134
time_in_draft.py                 28      0   100%
time_to_answer.py                28      0   100%
time_to_close.py                 38      2    95%   49, 55
time_to_first_response.py        71      3    96%   88-89, 121
time_to_merge.py                 12      0   100%
time_to_ready_for_review.py      15      3    80%   45-49
-----------------------------------------------------------
TOTAL                           760     71    91%

Required test coverage of 80% reached. Total coverage: 90.66%

Readiness Checklist

Author/Contributor

  • If documentation is needed for this change, has that been included in this pull request
  • run make lint and fix any issues that you have introduced
  • run make test and ensure you have test coverage for the lines you are introducing
  • If publishing new data to the public (scorecards, security scan results, code quality results, live dashboards, etc.), please request review from @jeffrey-luszcz

Reviewer

  • Label as either fix, documentation, enhancement, infrastructure, maintenance, or breaking

Copy link
Member

@jmeridth jmeridth left a comment

Choose a reason for hiding this comment

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

One suggestion up for discussion. Great catch on the misspellings. Thank you for adding this feature.

markdown_writer.py Show resolved Hide resolved
markdown_writer.py Show resolved Hide resolved
@jmeridth jmeridth merged commit baa4523 into github:main Oct 29, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants