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(tools): Add support for Stryker CLI #1259

Open
wants to merge 18 commits into
base: develop
Choose a base branch
from

Conversation

candoumbe
Copy link

This PR adds classes to use Stryker tool.
Classes were created using generate-tools target

I confirm that the pull-request:

  • Follows the contribution guidelines
  • Is based on my own work
  • Is in compliance with my employer

@candoumbe
Copy link
Author

@matkoch Are there any plan on reviewing/merging this PR ?

@matkoch matkoch force-pushed the develop branch 2 times, most recently from 786ee97 to 2949ed4 Compare March 19, 2024 17:45
@matkoch matkoch force-pushed the develop branch 2 times, most recently from 0719711 to 6fe1e3b Compare March 20, 2024 01:24
@matkoch matkoch force-pushed the develop branch 2 times, most recently from 6d02194 to 4bfee84 Compare June 13, 2024 21:33
@matkoch matkoch force-pushed the develop branch 4 times, most recently from 85998da to 01b2b6e Compare July 9, 2024 00:20
@matkoch matkoch force-pushed the develop branch 3 times, most recently from e652c9a to 6ed9e4d Compare July 14, 2024 18:27
@ITaluone
Copy link
Contributor

ITaluone commented Sep 12, 2024

@candoumbe You probably have to remove the generated file (see last bullet point: https://github.com/nuke-build/nuke/blob/develop/CONTRIBUTING.md#when-creating-a-pull-request)

@candoumbe
Copy link
Author

@ITaluone
I'll update the PR to support latest version of Stryker CLI and wil remove the generated file at that moment.

I'll switch the PR to draft

@candoumbe candoumbe marked this pull request as draft September 18, 2024 08:06
@candoumbe candoumbe marked this pull request as ready for review September 18, 2024 17:24
@candoumbe
Copy link
Author

@ITaluone I fixed the PR according to your comments

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.

6 participants