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.

Sign up for GitHub

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

Jump to bottom

fix: make schema pub, hide utils, fix indexing for rattler-build #1111

Merged
merged 8 commits into from
Feb 27, 2025
Merged

fix: make schema pub, hide utils, fix indexing for rattler-build #1111

merged 8 commits into from
Feb 27, 2025

Conversation

wolfv
Copy link
Contributor

@wolfv wolfv commented Feb 27, 2025 •
edited
Loading

This fixes some things I need in rattler-build:

  • makes schema public so that we can check the schema in rattler-build
  • remove the menuinst::utils from the public interface as this is purely internal
  • indexing was slightly broken which is fixed

@wolfv wolfv changed the title make schema pub, and hide utils fix: make schema pub, hide utils, fix indexing for rattler-build Feb 27, 2025
pavelzw
pavelzw approved these changes Feb 27, 2025
View reviewed changes
Copy link
Contributor

@pavelzw pavelzw left a comment

Choose a reason for hiding this comment

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

thanks!

@wolfv wolfv enabled auto-merge (squash) February 27, 2025 16:06
@wolfv wolfv merged commit 57301e9 into conda:main Feb 27, 2025
16 checks passed
@baszalmstra baszalmstra mentioned this pull request Feb 27, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pavelzw pavelzw pavelzw approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@wolfv @pavelzw