Skip to content

Fix publishprofile workflow - #567

Open
Akshay Kumar (kumaraksh1) wants to merge 7 commits into
masterfrom
users/kumaraksh/fix-publishprofile-workflow
Open

Fix publishprofile workflow#567
Akshay Kumar (kumaraksh1) wants to merge 7 commits into
masterfrom
users/kumaraksh/fix-publishprofile-workflow

Conversation

@kumaraksh1

Copy link
Copy Markdown
Collaborator

Separating untrusted PR validation from credentialed deployment to prevent PR-controlled


- name: Checkout from PR branch
uses: actions/checkout@v2
uses: actions/checkout@11d5960a326750d5838078e36cf38b85af677262 # v4

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

What version is this?

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Please update the PR description that this would mean this workflow cannot be used to test while submitting a PR.

Also, do we have any doc / readme that tells how to test one's PR. If yes, let's update there

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.

2 participants