LibGit2Sharp missing CredentialsProvider fix#2637
LibGit2Sharp missing CredentialsProvider fix#2637jmcouffin merged 3 commits intopyrevitlabs:developfrom
Conversation
- currently runs into an error if using a different LibGit2Sharp version not expecting CredentialsProvider property
|
Unable to perform a code review. You have run out of credits 😔 |
|
Hey @jmcouffin - I tried pushing a new PR with a small code fix, but ran into the above bot message: |
Don't worry about it. 2 emails down I got a reply from devlo, I will either get new credits or remove it |
There was a problem hiding this comment.
Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.
Comments suppressed due to low confidence (1)
pyrevitlib/pyrevit/coreutils/git.py:131
- Consider adding a test to verify that _make_clone_options correctly handles cases where CredentialsProvider is not supported by the API.
if username and password:
|
Thanks @dnenov |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25099+1655-wip |
Thanks for the super flash review @jmcouffin! I followed the instructions in the template and the formatting changed slightly, I hope that was OK. |
It is. Doing the formatting all at once on the codebase would be brutal. |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25101+0700-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25101+1501-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25104+0813-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25104+1015-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25104+1041-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25104+1127-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25105+0957-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25106+0734-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25106+1252-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25106+1322-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25106+1408-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25107+0621-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25116+1412-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25118+1113-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25119+1928-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25133+1246-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25136+0631-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25136+0639-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25140+1615-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25141+1904-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25143+0904-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25146+0559-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25149+1012-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25149+1943-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25153+1459-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25154+0858-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25154+0920-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25155+0904-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25162+1125-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25162+1306-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25162+2030-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25164+0700-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25164+1347-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25164+1420-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25164+1830-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25171+0757-wip |
|
📦 New work-in-progress (wip) builds are available for 5.2.0.25181+1313-wip |
|
📦 New public release are available for 5.2.0.25181+1425 |
LibGit2Sharp missing CredentialsProvider fix
Description
The purpose of this PR is to resolve a long-standing bug where using a different version of
LibGit2Sharplibrary will produce the following error when trying to install an extension from a private GitHub repository:The proposed fix is to check if the
CredentialsProviderproperty exists in the current API and only provide it then.Checklist
Before submitting your pull request, ensure the following requirements are met:
pipenv run black {source_file_or_directory}Related Issues
Additional Notes
LibGit2Sharpversion not expectingCredentialsProviderpropertyReviewers
@jmcouffin