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

docs: source file name same as output #371

Merged
merged 2 commits into from
Nov 15, 2024
Merged

Conversation

yasir-choudhry
Copy link
Contributor

Description

Tiny PR to change file name in docs.

When following the command line usage instructions the output file is called out_name.tar.gz but in the next step on the source machine it has my_env.tar.gz.
This is a problem if someone is following the instructions literally :)

If following the docs literally this file won’t exists
@yasir-choudhry yasir-choudhry requested review from xhochy and a team as code owners October 16, 2024 19:07
@conda-bot
Copy link
Contributor

We require contributors to sign our Contributor License Agreement and we don't have one on file for @yasir-choudhry.

In order for us to review and merge your code, please e-sign the Contributor License Agreement PDF. We then need to manually verify your signature, merge the PR (conda/infrastructure#1044), and ping the bot to refresh the PR.

@xhochy
Copy link
Collaborator

xhochy commented Oct 21, 2024

@yasir-choudhry Please ping again once the CLA has been signed.

@ericdill
Copy link
Contributor

@xhochy FYI the CLA just got merged into the infra repo earlier today

@jezdez
Copy link
Member

jezdez commented Nov 14, 2024

@conda-bot check

@conda-bot conda-bot added the cla-signed [bot] added once the contributor has signed the CLA label Nov 14, 2024
@xhochy xhochy merged commit 85ce5ee into conda:main Nov 15, 2024
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed [bot] added once the contributor has signed the CLA
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

5 participants