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

Update intro for invocation images #2957

Merged
merged 3 commits into from
Jan 6, 2024

Conversation

sbshah97
Copy link
Contributor

@sbshah97 sbshah97 commented Jan 5, 2024

What does this change

This updates the docs with a minor starter PR to resolve an issue in the docs.

What issue does it fix

Closes #2800

Notes for the reviewer

I have updated the PR based on my understand on what the issue required to be done. Happy to discuss more on this.

Checklist

  • Did you write documentation?
  • If this is your first pull request, please add your name to the bottom of our Contributors list. Thank you for making Porter better! 🙇‍♀️

Reviewer Checklist

  • Comment with /azp run test-porter-release if a magefile or build script was modified
  • Comment with /azp run porter-integration if it's a non-trivial PR

Signed-off-by: sbshah97 <sbs.191197@gmail.com>
Signed-off-by: sbshah97 <sbs.191197@gmail.com>
@sgettys sgettys enabled auto-merge January 5, 2024 18:33
Signed-off-by: Salman Shah <sbs.191197@gmail.com>
auto-merge was automatically disabled January 6, 2024 09:10

Head branch was pushed to by a user without write access

@sbshah97
Copy link
Contributor Author

sbshah97 commented Jan 6, 2024

/azp run porter-integration

Copy link

Commenter does not have sufficient privileges for PR 2957 in repo getporter/porter

@troy0820
Copy link
Member

troy0820 commented Jan 6, 2024

/azp run porter-integration

Copy link

No pipelines are associated with this pull request.

@sgettys sgettys enabled auto-merge January 6, 2024 22:41
@schristoff schristoff merged commit 037bd76 into getporter:main Jan 6, 2024
7 checks passed
@sbshah97 sbshah97 deleted the docs-update-invocation-images branch January 6, 2024 23:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Docs - intro-invocation-images.md update
4 participants