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

Updating documentation to describe ~= package specification #4930

Merged
merged 16 commits into from
Sep 19, 2023

Conversation

ryanskeith
Copy link
Contributor

@ryanskeith ryanskeith commented Jul 10, 2023

Description

Attempting to address this issue #4553 .

Also, made the examples more readable by making them in table format.

Checklist - did you ...

  • Add a file to the news directory (using the template) for the next release's release notes?
  • Add / update outdated documentation?

@conda-bot conda-bot added the cla-signed [bot] added once the contributor has signed the CLA label Jul 10, 2023
@ryanskeith ryanskeith changed the title Updating documentation to describe .conda packages Updating documentation to describe ~= Jul 11, 2023
@ryanskeith ryanskeith changed the title Updating documentation to describe ~= Updating documentation to describe ~= package specification Jul 11, 2023
@ryanskeith ryanskeith changed the title Updating documentation to describe ~= package specification [DRAFT] Updating documentation to describe ~= package specification Jul 11, 2023
@ryanskeith
Copy link
Contributor Author

I just need to check how the compatibility clause gets translated in a meta.yaml to make the documentation correct.

@beeankha beeankha marked this pull request as draft July 13, 2023 19:19
@ryanskeith
Copy link
Contributor Author

ryanskeith commented Sep 15, 2023

BEFORE SHOT:
Screenshot 2023-09-15 at 1 51 27 PM
AFTER SHOT:
Screenshot 2023-09-15 at 1 51 41 PM

@ryanskeith ryanskeith changed the title [DRAFT] Updating documentation to describe ~= package specification Updating documentation to describe ~= package specification Sep 15, 2023
beeankha
beeankha previously approved these changes Sep 16, 2023
Copy link
Member

@beeankha beeankha left a comment

Choose a reason for hiding this comment

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

These changes look great!

Co-authored-by: Jannis Leidel <jannis@leidel.info>
Copy link
Contributor

@kathatherine kathatherine left a comment

Choose a reason for hiding this comment

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

Some suggestions and a few questions/comments. Thanks, Ryan!

docs/source/resources/package-spec.rst Outdated Show resolved Hide resolved
docs/source/resources/package-spec.rst Outdated Show resolved Hide resolved
docs/source/resources/package-spec.rst Outdated Show resolved Hide resolved
docs/source/resources/package-spec.rst Outdated Show resolved Hide resolved
docs/source/resources/package-spec.rst Outdated Show resolved Hide resolved
docs/source/resources/package-spec.rst Outdated Show resolved Hide resolved
docs/source/resources/package-spec.rst Outdated Show resolved Hide resolved
ryanskeith and others added 6 commits September 18, 2023 13:21
Co-authored-by: Katherine Kinnaman <kkinnaman@anaconda.com>
Co-authored-by: Katherine Kinnaman <kkinnaman@anaconda.com>
Co-authored-by: Katherine Kinnaman <kkinnaman@anaconda.com>
Co-authored-by: Katherine Kinnaman <kkinnaman@anaconda.com>
Co-authored-by: Katherine Kinnaman <kkinnaman@anaconda.com>
Copy link
Contributor

@kathatherine kathatherine left a comment

Choose a reason for hiding this comment

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

A few more things

docs/source/resources/package-spec.rst Show resolved Hide resolved
docs/source/resources/package-spec.rst Outdated Show resolved Hide resolved
@ryanskeith ryanskeith marked this pull request as ready for review September 18, 2023 22:04
Copy link
Contributor

@kathatherine kathatherine left a comment

Choose a reason for hiding this comment

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

Thanks!

@jezdez jezdez merged commit 3ba167b into conda:main Sep 19, 2023
@ryanskeith ryanskeith deleted the issue/4553 branch September 19, 2023 19:43
@kenodegard kenodegard mentioned this pull request Sep 26, 2023
36 tasks
@github-actions github-actions bot added the locked [bot] locked due to inactivity label Sep 19, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 19, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla-signed [bot] added once the contributor has signed the CLA locked [bot] locked due to inactivity
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

5 participants