Skip to content

LibGit2Sharp missing CredentialsProvider fix#2637

Merged
jmcouffin merged 3 commits intopyrevitlabs:developfrom
dnenov:git-install-privte-fix
Apr 9, 2025
Merged

LibGit2Sharp missing CredentialsProvider fix#2637
jmcouffin merged 3 commits intopyrevitlabs:developfrom
dnenov:git-install-privte-fix

Conversation

@dnenov
Copy link
Copy Markdown
Contributor

@dnenov dnenov commented Apr 9, 2025

LibGit2Sharp missing CredentialsProvider fix

Description

The purpose of this PR is to resolve a long-standing bug where using a different version of LibGit2Sharp library will produce the following error when trying to install an extension from a private GitHub repository:

image

The proposed fix is to check if the CredentialsProvider property exists in the current API and only provide it then.


Checklist

Before submitting your pull request, ensure the following requirements are met:

  • Code follows the PEP 8 style guide.
  • Code has been formatted with Black using the command:
    pipenv run black {source_file_or_directory}
  • Changes are tested and verified to work as expected.

Related Issues

  • n/a

Additional Notes

  • currently runs into an error if using a different LibGit2Sharp version not expecting CredentialsProvider property

Reviewers

@jmcouffin

dnenov added 2 commits April 9, 2025 15:20
- currently runs into an error if using a different LibGit2Sharp version not expecting CredentialsProvider property
@devloai
Copy link
Copy Markdown
Contributor

devloai bot commented Apr 9, 2025

Unable to perform a code review. You have run out of credits 😔
You can buy additional credits from the subscription page.

@dnenov
Copy link
Copy Markdown
Contributor Author

dnenov commented Apr 9, 2025

Hey @jmcouffin - I tried pushing a new PR with a small code fix, but ran into the above bot message: Unable to perform a code review. You have run out of credits 😔 You can buy additional credits from the [subscription page](https://devlo.ai/settings). Just flagging it up, I am not sure if there is a policy I need to abide by.

@jmcouffin
Copy link
Copy Markdown
Contributor

Hey @jmcouffin - I tried pushing a new PR with a small code fix, but ran into the above bot message: Unable to perform a code review. You have run out of credits 😔 You can buy additional credits from the [subscription page](https://devlo.ai/settings). Just flagging it up, I am not sure if there is a policy I need to abide by.

Don't worry about it. 2 emails down I got a reply from devlo, I will either get new credits or remove it

@jmcouffin jmcouffin requested a review from Copilot April 9, 2025 16:49
@jmcouffin jmcouffin self-assigned this Apr 9, 2025
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

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:

@jmcouffin jmcouffin added the Extensions Issues related to pyRevit extensions [subsystem] label Apr 9, 2025
@jmcouffin jmcouffin merged commit 994705a into pyrevitlabs:develop Apr 9, 2025
@jmcouffin
Copy link
Copy Markdown
Contributor

Thanks @dnenov
The little things that matter most. That's a nice one. And clean!

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 9, 2025

📦 New work-in-progress (wip) builds are available for 5.1.0.25099+1655-wip

@dnenov
Copy link
Copy Markdown
Contributor Author

dnenov commented Apr 10, 2025

Thanks @dnenov The little things that matter most. That's a nice one. And clean!

Thanks for the super flash review @jmcouffin! I followed the instructions in the template and the formatting changed slightly, I hope that was OK.

@jmcouffin
Copy link
Copy Markdown
Contributor

I hope that was OK.

It is. Doing the formatting all at once on the codebase would be brutal.

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25101+0700-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25101+1501-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25104+0813-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25104+1015-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25104+1041-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25104+1127-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25105+0957-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25106+0734-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25106+1252-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25106+1322-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25106+1408-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25107+0621-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25116+1412-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25118+1113-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25119+1928-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25133+1246-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25136+0631-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25136+0639-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25140+1615-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25141+1904-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25143+0904-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25146+0559-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25149+1012-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25149+1943-wip

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jun 2, 2025

📦 New work-in-progress (wip) builds are available for 5.1.0.25153+1459-wip

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jun 3, 2025

📦 New work-in-progress (wip) builds are available for 5.1.0.25154+0858-wip

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jun 3, 2025

📦 New work-in-progress (wip) builds are available for 5.1.0.25154+0920-wip

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jun 4, 2025

📦 New work-in-progress (wip) builds are available for 5.1.0.25155+0904-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25162+1125-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25162+1306-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25162+2030-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25164+0700-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25164+1347-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25164+1420-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25164+1830-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25171+0757-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.2.0.25181+1313-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New public release are available for 5.2.0.25181+1425

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Extensions Issues related to pyRevit extensions [subsystem]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants