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: clarify Skipped and Omitted in enhanced depends #13087

Merged
merged 3 commits into from
May 28, 2024

Conversation

oliverdain
Copy link
Contributor

docs: update the enhanced depends documentation

It wasn't clear to me what Omitted and Skipped meant. I'm still not sure I have this correct but I'm going off advice I got in the Argo Slack.

@oliverdain oliverdain changed the title Draft: Cleaned up the enhanced depends docs a little. docs: Cleaned up the enhanced depends docs a little. May 24, 2024
Signed-off-by: Oliver Dain <oliver@dains.org>
Signed-off-by: Oliver Dain <oliver@dains.org>
Signed-off-by: Oliver Dain <oliver@dains.org>
@Joibel Joibel self-assigned this May 24, 2024
@Joibel Joibel added the area/docs Incorrect, missing, or mistakes in docs label May 24, 2024
@agilgur5 agilgur5 changed the title docs: Cleaned up the enhanced depends docs a little. docs: clarify Skipped and Omitted in enhanced depends May 27, 2024
@Joibel Joibel merged commit f492ea3 into argoproj:main May 28, 2024
17 checks passed
@Joibel
Copy link
Member

Joibel commented May 28, 2024

Thanks @oliverdain for your contribution.

Copy link
Member

@agilgur5 agilgur5 left a comment

Choose a reason for hiding this comment

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

I left some grammatical corrections below, but this can also be improved and simplified, per our style guide.

I think this whole page could use a copy-edit though (like some of my previous docs changes), so I can make those changes as part of a larger rewrite. EDIT: See #13110

docs/enhanced-depends-logic.md Show resolved Hide resolved
docs/enhanced-depends-logic.md Show resolved Hide resolved
@agilgur5 agilgur5 added this to the v3.5.x patches milestone May 29, 2024
agilgur5 pushed a commit that referenced this pull request Jun 2, 2024
Signed-off-by: Oliver Dain <oliver@dains.org>
(cherry picked from commit f492ea3)
agilgur5 pushed a commit that referenced this pull request Jun 17, 2024
Signed-off-by: Oliver Dain <oliver@dains.org>
(cherry picked from commit f492ea3)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/docs Incorrect, missing, or mistakes in docs area/templates/dag
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants