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

Transition from using setup.py to pyproject.toml to specify project metadata #766

Merged
merged 13 commits into from
Mar 5, 2024

Conversation

R-Palazzo
Copy link
Contributor

CU-86azg1ghp
Resolve #763

@R-Palazzo R-Palazzo requested a review from a team as a code owner March 4, 2024 10:49
@sdv-team
Copy link
Contributor

sdv-team commented Mar 4, 2024

@R-Palazzo R-Palazzo removed the request for review from a team March 4, 2024 10:49
pyproject.toml Outdated Show resolved Hide resolved
@codecov-commenter
Copy link

codecov-commenter commented Mar 4, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (1ae11e8) to head (5d8814a).

❗ Current head 5d8814a differs from pull request most recent head f332d2d. Consider uploading reports for the commit f332d2d to get more accurate results

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

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #766   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           18        16    -2     
  Lines         2063      1993   -70     
=========================================
- Hits          2063      1993   -70     

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

rdt/__init__.py Outdated
setattr(addon_target, addon_name, addon)


_find_addons()
Copy link
Member

Choose a reason for hiding this comment

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

This code has to stay! 😅 Otherwise enterprise addons won't work

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good catch, thanks! Updated in f332d2d

Copy link
Member

@pvk-developer pvk-developer left a comment

Choose a reason for hiding this comment

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

LGTM ;)!

Copy link
Contributor

@amontanez24 amontanez24 left a comment

Choose a reason for hiding this comment

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

LGTM!

@R-Palazzo R-Palazzo merged commit c84e5df into main Mar 5, 2024
45 checks passed
@R-Palazzo R-Palazzo deleted the issue-763-pyproject branch March 5, 2024 07:59
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.

Transition from using setup.py to pyproject.toml to specify project metadata
5 participants