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: create parameter file #25

Merged
merged 2 commits into from
Jan 16, 2025
Merged

docs: create parameter file #25

merged 2 commits into from
Jan 16, 2025

Conversation

hknutsen
Copy link
Member

Recommend passing parameters using a parameter file rathen than as inline values.

@hknutsen
docs: create parameter file
d1adc41 
Recommend passing parameters using a parameter file rathen than as inline values.
@hknutsen hknutsen self-assigned this Jan 15, 2025
@hknutsen hknutsen marked this pull request as ready for review January 15, 2025 07:07
@hknutsen hknutsen requested a review from a team as a code owner January 15, 2025 07:07
@hknutsen
Copy link
Member Author

hknutsen commented Jan 15, 2025 •
edited
Loading

@equinor/ops-actions-maintainers Note that this PR contains two commits:

  • In the first commit, I placed the tip in the "Usage" section, right below the step where we show how to pass a parameter as an inline value.
  • In the second (current) commit, I placed the tip in the "Parameters" section, right under the table containing detailed information about each available parameter.

What do you think is the best placement?

@sefornes sefornes merged commit 7abd15b into main Jan 16, 2025
1 check passed
@sefornes sefornes deleted the docs/create-parameter-file branch January 16, 2025 08:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sefornes sefornes sefornes approved these changes

Assignees

@hknutsen hknutsen

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@hknutsen @sefornes