Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: introduce default interactive mode #102

Merged
merged 3 commits into from
Mar 12, 2024
Merged

Conversation

slandelle
Copy link
Member

Motivation:

We want to align the maven and the gradle plugins behavior.

In particular, we want it to be interactive by default, with a simulation selection prompt.

@slandelle slandelle requested review from Isammoc and guilgaly March 5, 2024 22:31
@@ -17,16 +17,13 @@ final class GatlingPlugin implements Plugin<Project> {

public static def GATLING_RECORDER_TASK_NAME = 'gatlingRecorder'

static String GATLING_TASK_NAME_PREFIX = "$GATLING_RUN_TASK_NAME-"
Copy link
Member Author

Choose a reason for hiding this comment

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

gatlingRun-FQCN is dropped in favor of gatling --simulation=FQCN

@slandelle slandelle force-pushed the feat-interactive-mode branch 4 times, most recently from e797bb8 to b5fae64 Compare March 5, 2024 22:48
@slandelle slandelle requested a review from Isammoc March 7, 2024 04:59
@slandelle slandelle requested review from Isammoc and guilgaly March 12, 2024 15:05
@slandelle slandelle force-pushed the feat-interactive-mode branch from 747e8fe to 763da7a Compare March 12, 2024 15:07
Motivation:

We want to align the maven and the gradle plugins behavior.

In particular, we want it to be interactive by default, with a simulation selection prompt.
@slandelle slandelle force-pushed the feat-interactive-mode branch 2 times, most recently from 0fd1731 to e7db854 Compare March 12, 2024 15:39
Copy link
Member

@Isammoc Isammoc left a comment

Choose a reason for hiding this comment

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

nits

Wonderful 👏

@slandelle slandelle force-pushed the feat-interactive-mode branch from 2fe7fc0 to a7d4dc0 Compare March 12, 2024 19:04
@slandelle slandelle merged commit 9474850 into main Mar 12, 2024
1 check passed
@slandelle slandelle deleted the feat-interactive-mode branch March 12, 2024 19:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants