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

Update parcel.rst #317

Merged
merged 1 commit into from
Apr 7, 2024
Merged

Update parcel.rst #317

merged 1 commit into from
Apr 7, 2024

Conversation

bkreider
Copy link
Contributor

The documentation was wrong. It called the parameter --parcel-distribution but it should be --parcel-distro.

Description

See definition in cli.py:

    parser.add_argument("--parcel-distro", default=None,
                        help="(Parcels only) The distribution type for the parcel. The "
                        "default value is 'el7'. This value cannot have any hyphens.")

Checklist - did you ...

  • [x ] Add / update outdated documentation?

@bkreider
Update parcel.rst
0ba3f32 
The documentation was wrong.  It called the parameter `--parcel-distribution` but it should be `--parcel-distro`.
@bkreider bkreider requested review from xhochy and a team as code owners March 27, 2024 23:50
@conda-bot conda-bot added the cla-signed [bot] added once the contributor has signed the CLA label Mar 27, 2024
@xhochy xhochy merged commit 5f8c9bd into main Apr 7, 2024
21 checks passed
@xhochy xhochy deleted the bkreider-patch-1 branch April 7, 2024 14:54
ayurchuk pushed a commit to ayurchuk/conda-pack that referenced this pull request Apr 21, 2024
@bkreider
Update parcel.rst (conda#317)
bda6b07 
The documentation was wrong.  It called the parameter `--parcel-distribution` but it should be `--parcel-distro`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xhochy xhochy xhochy approved these changes

Assignees
No one assigned
Labels
cla-signed [bot] added once the contributor has signed the CLA
Projects
🔎 Review
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@bkreider @xhochy @conda-bot