Skip to content

Convert action to lazyAction#7974

Closed
ChristopherDedominici wants to merge 2 commits intoadd-inline-action-runtime-checkfrom
add-inline-action-runtime-check-rename
Closed

Convert action to lazyAction#7974
ChristopherDedominici wants to merge 2 commits intoadd-inline-action-runtime-checkfrom
add-inline-action-runtime-check-rename

Conversation

@ChristopherDedominici
Copy link
Copy Markdown
Contributor

@ChristopherDedominici ChristopherDedominici commented Feb 17, 2026

This PR is created on top of this one: #7950

In this PR, the ONLY changes are the renames from action to lazyAction.
These affects:

  • src code
  • tests
  • comments
  • types

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Feb 17, 2026 •
edited
Loading

⚠️ No Changeset found

Latest commit: 919b96f

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@ChristopherDedominici ChristopherDedominici mentioned this pull request Feb 18, 2026
Merged
@github-project-automation github-project-automation Bot moved this to Backlog in Hardhat Feb 19, 2026
@ChristopherDedominici ChristopherDedominici moved this from Backlog to In Review in Hardhat Feb 19, 2026
@ChristopherDedominici ChristopherDedominici mentioned this pull request Feb 19, 2026
Merged
alcuadrado
alcuadrado reviewed Feb 21, 2026
View reviewed changes
Comment thread v-next/example-project/hardhat.config.ts
const exampleEmptySubtask = task(["empty", "task"])
.setDescription("An example empty subtask task")
.setAction(async () => ({
.setLazyAction(async () => ({
Copy link
Copy Markdown
Member

@alcuadrado alcuadrado Feb 21, 2026

Choose a reason for hiding this comment

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

oh, this was confusing on my end, sorry. I think from the POV of a user we should change it. I was referring only to the field in the builder and definitions.

Copy link
Copy Markdown
Member

@alcuadrado alcuadrado Feb 21, 2026

Choose a reason for hiding this comment

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

Because this does look like a breaking change without a real benefit otherwise

@github-project-automation github-project-automation Bot moved this from In Review to Done in Hardhat Feb 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alcuadrado alcuadrado alcuadrado left review comments

Assignees

@ChristopherDedominici ChristopherDedominici

Labels

None yet

Projects

Hardhat
Archived in project

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ChristopherDedominici @alcuadrado @kanej