Skip to content

feat: bump checkout to v7 w/ allow-unsafe-pr-checkout arg#6242

Open
krzysztofczyz-da wants to merge 2 commits into
mainfrom
kczyz/pull_request_target-save
Open

feat: bump checkout to v7 w/ allow-unsafe-pr-checkout arg#6242
krzysztofczyz-da wants to merge 2 commits into
mainfrom
kczyz/pull_request_target-save

Conversation

@krzysztofczyz-da

Copy link
Copy Markdown
Contributor

No description provided.

Signed-off-by: krzysztofczyz-da <krzysztof.czyz@digitalasset.com>
@krzysztofczyz-da krzysztofczyz-da added the static Used to label PRs for which static tests suffice label Jul 6, 2026

- name: Check out repository code
uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
uses: actions/checkout@9c091bb21b7c1c1d1991bb908d89e4e9dddfe3e0 # v7.0.0

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Any reason not to bump all our current uses: actions/checkout?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

offtopic: on internal we're currently on v4.2.2 🤡

(keeping it consistent within the same repo still sounds better to me)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Agreed, let's move everything to v7

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done

Signed-off-by: krzysztofczyz-da <krzysztof.czyz@digitalasset.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

static Used to label PRs for which static tests suffice

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants