Skip to content

Conversation

@Ni-g-3l
Copy link
Contributor

@Ni-g-3l Ni-g-3l commented Jun 10, 2025

This PR is the same as : #1872 which I closed by error...

Sorry for inconvenient..

Old description

Hello

As suggest here I implemented the support of entrypoints in order to make the process of adding plugin to REZ easier.

Closes #1661

Feel free to add feedback !

Have a great day

@Ni-g-3l Ni-g-3l requested a review from a team as a code owner June 10, 2025 18:58
@Ni-g-3l Ni-g-3l force-pushed the feat/register_plugin_from_entry_points branch from eee918a to 1d532d1 Compare June 10, 2025 18:59
@codecov
Copy link

codecov bot commented Jun 10, 2025

Codecov Report

Attention: Patch coverage is 68.51852% with 17 lines in your changes missing coverage. Please review.

Project coverage is 60.09%. Comparing base (4708523) to head (f117b75).
Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
src/rez/plugin_managers.py 68.51% 12 Missing and 5 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1989      +/-   ##
==========================================
+ Coverage   60.02%   60.09%   +0.07%     
==========================================
  Files         163      163              
  Lines       20118    20151      +33     
  Branches     3506     3511       +5     
==========================================
+ Hits        12075    12110      +35     
+ Misses       7223     7220       -3     
- Partials      820      821       +1     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Ni-g-3l Ni-g-3l force-pushed the feat/register_plugin_from_entry_points branch from 1d532d1 to f117b75 Compare June 10, 2025 19:44
@JeanChristopheMorinPerso
Copy link
Member

@Ni-g-3l How did you recreate the branch? Some of my changes seem to be there, but my commits are not there... I'm confused.

@JeanChristopheMorinPerso
Copy link
Member

I pushed my local branch to https://github.com/JeanChristopheMorinPerso/rez/tree/feat/register_plugin_from_entry_points_jc. Can you take it re-create a PR from it please? That will put us in the same state we were before.

@JeanChristopheMorinPerso JeanChristopheMorinPerso added this to the Next milestone Jun 21, 2025
@JeanChristopheMorinPerso JeanChristopheMorinPerso changed the title Feat/register plugin from entry points Register plugins from entry points Jun 21, 2025
@Ni-g-3l Ni-g-3l closed this Jun 22, 2025
@Ni-g-3l Ni-g-3l force-pushed the feat/register_plugin_from_entry_points branch from f117b75 to dac6bcb Compare June 22, 2025 18:37
@Ni-g-3l Ni-g-3l deleted the feat/register_plugin_from_entry_points branch June 22, 2025 18:39
@Ni-g-3l Ni-g-3l restored the feat/register_plugin_from_entry_points branch June 22, 2025 18:41
@Ni-g-3l Ni-g-3l deleted the feat/register_plugin_from_entry_points branch June 22, 2025 18:41
@Ni-g-3l
Copy link
Contributor Author

Ni-g-3l commented Jun 22, 2025

Sorry again ! Here is the new PR : #1991 (The last one I hope) with all of your commits and conflict fixed with master !

Thanks a lot !

@JeanChristopheMorinPerso
Copy link
Member

Great, thanks a lot!

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.

Simplify plugin development by supporting entrypoint based plugins

2 participants