Skip to content

Move script to get refs from PR body for reuse#1589

Merged
erikburt merged 2 commits into
mainfrom
feature/get-ref-from-pr-body
Jul 16, 2026
Merged

Move script to get refs from PR body for reuse#1589
erikburt merged 2 commits into
mainfrom
feature/get-ref-from-pr-body

Conversation

@dhaidashenko

@dhaidashenko dhaidashenko commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

@dhaidashenko
dhaidashenko requested a review from a team as a code owner July 16, 2026 11:44
@github-actions

Copy link
Copy Markdown
Contributor

👋 dhaidashenko, thanks for creating this pull request!

To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team.

Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks!

@erikburt erikburt left a comment

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.

How is this intended to be used? Should it be made into an action?

@dhaidashenko

Copy link
Copy Markdown
Contributor Author

How is this intended to be used? Should it be made into an action?

The script can be imported into another action to get access to refs. Chainlink-evm and chainlink-common will use it in their respective actions to check whether the Chainlink build breaks on the current commit.

I've followed the pattern used with another script here.
Let me know if actions are prefered way of reuse

@dhaidashenko
dhaidashenko requested a review from erikburt July 16, 2026 15:12
@erikburt

Copy link
Copy Markdown
Contributor

The script can be imported into another action to get access to refs. Chainlink-evm and chainlink-common will use it in their respective actions to check whether the Chainlink build breaks on the current commit.

I've followed the pattern used with another script here. Let me know if actions are prefered way of reuse

Yes, reusable actions are the preferred way. We can merge this now to unblock you though.

@erikburt
erikburt merged commit 79a5c88 into main Jul 16, 2026
18 checks passed
@erikburt
erikburt deleted the feature/get-ref-from-pr-body branch July 16, 2026 15:21
@erikburt

Copy link
Copy Markdown
Contributor

Created DX-4670 to track the work on moving this to reusable GHAs instead of clone+invoke style usage.

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