Skip to content

Add has_discussions to AuthenticatedUser and Repository classes#3020

Merged
EnricoMi merged 4 commits intoPyGithub:mainfrom
cwlls:main
Aug 26, 2024
Merged

Add has_discussions to AuthenticatedUser and Repository classes#3020
EnricoMi merged 4 commits intoPyGithub:mainfrom
cwlls:main

Conversation

@cwlls
Copy link
Copy Markdown

@cwlls cwlls commented Aug 16, 2024

The has_discussions feature is exposed in the Github API and is identical in the way it works to something like has_wiki. Following a couple previous PR attempts by @amnuts and @heitorPB as well as the existing code for has_wiki I added has_discussions as well as the test cases.

Fixes #2733 and #2994

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Aug 16, 2024

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.73%. Comparing base (6d60b75) to head (e7b80e3).
Report is 33 commits behind head on main.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@           Coverage Diff            @@
##             main    #3020    +/-   ##
========================================
  Coverage   96.73%   96.73%            
========================================
  Files         149      150     +1     
  Lines       15061    15254   +193     
========================================
+ Hits        14569    14756   +187     
- Misses        492      498     +6     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.


🚨 Try these New Features:

@cwlls
Copy link
Copy Markdown
Author

cwlls commented Aug 16, 2024

Sorry for the messy set of commits. I finally checked out the repo and ran the tests locally to figure out what was going wrong with the testing Replay misses.

@amnuts
Copy link
Copy Markdown
Contributor

amnuts commented Aug 18, 2024

If this is approved and merged in, then close out mine at the same time: #2781

Copy link
Copy Markdown
Collaborator

@EnricoMi EnricoMi left a comment

Choose a reason for hiding this comment

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

LGTM!

@EnricoMi EnricoMi changed the title Add has_discussions to AuthenticatedUser and Repository classes Add has_discussions to AuthenticatedUser and Repository classes Aug 26, 2024
@EnricoMi EnricoMi added this pull request to the merge queue Aug 26, 2024
Merged via the queue into PyGithub:main with commit 7522416 Aug 26, 2024
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.

[Feature request] Add support for repo.has_discussions

4 participants