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

Consider packages.conda for index update and channel mirroring #638

Merged
merged 5 commits into from
Aug 22, 2023

Conversation

YYYasin19
Copy link
Collaborator

As reported in #599: The current implementation neglects packages that are under packages.conda in the repodata.json files.
This PR should close #599; I hope I haven't missed any other places.

There's one reference to utils/repodata_compare.py but that file seems to be unused right now.

@codecov-commenter
Copy link

codecov-commenter commented Jun 1, 2023

Codecov Report

Patch coverage: 92.30% and project coverage change: +0.06% 🎉

Comparison is base (2f819d4) 83.26% compared to head (ec4f67b) 83.33%.
Report is 2 commits behind head on main.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #638      +/-   ##
==========================================
+ Coverage   83.26%   83.33%   +0.06%     
==========================================
  Files          78       78              
  Lines        6186     6205      +19     
==========================================
+ Hits         5151     5171      +20     
+ Misses       1035     1034       -1     
Files Changed Coverage Δ
quetz/main.py 86.46% <66.66%> (ø)
quetz/authorization.py 93.82% <95.00%> (+0.10%) ⬆️
quetz/tasks/indexing.py 91.50% <100.00%> (ø)
quetz/tasks/mirror.py 89.31% <100.00%> (ø)

... and 2 files with indirect coverage changes

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

@janjagusch janjagusch added the enhancement New feature or request label Jun 1, 2023
@janjagusch janjagusch changed the title consider packages.conda for index update and channel mirroring Consider packages.conda for index update and channel mirroring Jun 2, 2023
Copy link
Collaborator

@janjagusch janjagusch left a comment

Choose a reason for hiding this comment

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

Nice! What would be a good way to test this?

@janjagusch
Copy link
Collaborator

@YYYasin19, can you please rebase with the latest commit on main?

@YYYasin19
Copy link
Collaborator Author

Nice! What would be a good way to test this?

I'll look into how the current tests are working and implement some additional ones for .conda packages.

@wolfv wolfv merged commit b2eb701 into mamba-org:main Aug 22, 2023
9 checks passed
@wolfv
Copy link
Member

wolfv commented Aug 22, 2023

Thank you @YYYasin19!

@janjagusch janjagusch added bug Something isn't working and removed enhancement New feature or request labels Sep 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support .conda package format
5 participants