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

docs: Minor expansion of application order docs #3201

Merged
merged 1 commit into from
Aug 29, 2023
Merged

docs: Minor expansion of application order docs #3201

merged 1 commit into from
Aug 29, 2023

Conversation

halostatue
Copy link
Collaborator

The discussion in #3198 suggests some confusion about the order of application. This adds a little more context.

@halostatue halostatue added the documentation This affects documentation, not code label Aug 29, 2023
@halostatue halostatue self-assigned this Aug 29, 2023
arrrgi
arrrgi approved these changes Aug 29, 2023
View reviewed changes
Copy link

@arrrgi arrrgi left a comment

Choose a reason for hiding this comment

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

LGTM and definitely clears up the use case we've discussed under #3198

@halostatue
docs: Minor expansion of application order docs
3af9ae4 
The discussion in twpayne#3198 suggests some confusion about the order of
application. This adds a little more context.
twpayne
twpayne approved these changes Aug 29, 2023
View reviewed changes
Copy link
Owner

@twpayne twpayne left a comment

Choose a reason for hiding this comment

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

Thank you!

@halostatue halostatue merged commit fe903d4 into twpayne:master Aug 29, 2023
18 checks passed
@halostatue halostatue deleted the run-order-clarifications branch August 29, 2023 13:22
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 10, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@arrrgi arrrgi arrrgi approved these changes

@twpayne twpayne twpayne approved these changes

Assignees

@halostatue halostatue

Labels
documentation This affects documentation, not code
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@halostatue @twpayne @arrrgi