Skip to content

refactor: move upload from rattler-build to rattler#1386

Merged
Hofer-Julian merged 11 commits intoconda:mainfrom
magentaqin:refactor/upload-alignment
Jul 9, 2025
Merged

refactor: move upload from rattler-build to rattler#1386
Hofer-Julian merged 11 commits intoconda:mainfrom
magentaqin:refactor/upload-alignment

Conversation

@magentaqin
Copy link
Contributor

@magentaqin magentaqin commented Jun 10, 2025

Description

Create new crate rattler_upload to solve issue Align pixi upload with rattler-build upload

@magentaqin magentaqin force-pushed the refactor/upload-alignment branch 2 times, most recently from 13baea5 to 5a75312 Compare July 2, 2025 07:43
@magentaqin magentaqin force-pushed the refactor/upload-alignment branch from b36c380 to a2a01c3 Compare July 9, 2025 08:42
@magentaqin magentaqin changed the title Move upload in rattler-build to rattler refactor: move upload from rattler-build to rattler Jul 9, 2025
@magentaqin magentaqin marked this pull request as ready for review July 9, 2025 09:37
Copy link
Collaborator

@Hofer-Julian Hofer-Julian left a comment

Choose a reason for hiding this comment

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

Only had one small comment, apart from that it looks good!

AnacondaData, ArtifactoryData, CondaForgeData, PrefixData, QuetzData, ServerType, UploadOpts,
};

/// Upload.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can you please add a more descriptive docstring here?

Doesn't have to be long, but one word is not enough :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Got it!

Copy link
Collaborator

@Hofer-Julian Hofer-Julian left a comment

Choose a reason for hiding this comment

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

Great work 🚀

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.

2 participants