Skip to content

Conversation

@clrudolphi
Copy link
Contributor

🤔 What's changed?

Enhanced the Define Steps command to support generating method stubs that are async.
This honors the reqnroll.json Trace setting introduced in Reqnroll PR#876.
The DefineStep dialog box now has a check box to indicate whether to use async methods.

⚡️ What's your motivation?

Developer productivity by eliminating the need to edit the generated stubs to support async.

🏷️ What kind of change is this?

  • ⚡ New feature (non-breaking change which adds new behaviour)

♻️ Anything particular you want feedback on?

Haven't changed any documentation yet.

📋 Checklist:

  • I've changed the behaviour of the code
    • I have added/updated tests to cover my changes.
  • My change requires a change to the documentation.
    • I have updated the documentation accordingly.
  • Users should know about my change
    • I have added an entry to the "[vNext]" section of the CHANGELOG, linking to this pull request & included my GitHub handle to the release contributors list.

This text was originally taken from the template of the Cucumber project, then edited by hand. You can modify the template here.

@clrudolphi clrudolphi requested a review from gasparnagy October 5, 2025 17:23
@clrudolphi clrudolphi mentioned this pull request Oct 6, 2025
Copy link
Contributor

@gasparnagy gasparnagy left a comment

Choose a reason for hiding this comment

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

We need to rework this according to the final solution in reqnroll/Reqnroll#876 🙁

@clrudolphi clrudolphi added the parked We decided to delay dealing with this label Oct 22, 2025
@clrudolphi
Copy link
Contributor Author

Parking this; keeping it around temporarily for reference.
Going to break this into two parts: first one will do the generation (respecting the configured setting in reqnroll.json).
A second PR will provide a UI (if we ever decide such is needed).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

parked We decided to delay dealing with this

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants