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

Update stryker monorepo (major) #140

Merged
merged 3 commits into from Jan 25, 2021
Merged

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Oct 7, 2020

WhiteSource Renovate

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
@stryker-mutator/core ^3.3.1 -> ^4.3.1 age adoption passing confidence
@stryker-mutator/javascript-mutator ^3.3.1 -> ^4.0.0 age adoption passing confidence

Release Notes

stryker-mutator/stryker

v4.3.1

Compare Source

Note: Version bump only for package stryker-parent

v4.2.0

Compare Source

Bug Fixes
Features

4.1.2 (2020-11-06)

Bug Fixes
  • peerDeps: update core in peerDependencies (045dbc3)

4.1.1 (2020-11-05)

Bug Fixes

v4.1.2

Compare Source

Bug Fixes
  • peerDeps: update core in peerDependencies (045dbc3)

v4.1.1

Compare Source

Bug Fixes

v4.1.0

Compare Source

Bug Fixes
Features

v4.0.0

Compare Source

Bug Fixes

Renovate configuration

📅 Schedule: At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

♻️ Rebasing: Renovate will not automatically rebase this PR, because other commits have been found.

👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.


  • If you want to rebase/retry this PR, check this box

This PR has been generated by WhiteSource Renovate. View repository job log here.

@renovate renovate bot force-pushed the renovate/major-stryker-monorepo branch from da8db10 to d175a6a Compare October 7, 2020 08:33
@renovate renovate bot changed the title Update @stryker-mutator/javascript-mutator -> 4 - devDependencies Update stryker monorepo -> 4 - devDependencies (major) Oct 7, 2020
@karfau
Copy link
Member

karfau commented Oct 7, 2020

I think landing this requires additional checks, since we are currently only running stryker on master.

If the improved performance allows it we might consider adding stryker to branch checks.

@renovate renovate bot force-pushed the renovate/major-stryker-monorepo branch from d175a6a to e3a35ed Compare October 30, 2020 07:40
@renovate renovate bot changed the title Update stryker monorepo -> 4 - devDependencies (major) Update stryker monorepo (major) Oct 30, 2020
@renovate renovate bot force-pushed the renovate/major-stryker-monorepo branch 4 times, most recently from cc4782c to d9768f6 Compare November 6, 2020 07:52
@renovate renovate bot force-pushed the renovate/major-stryker-monorepo branch 2 times, most recently from 13da63b to 8d086a8 Compare December 15, 2020 11:38
@renovate renovate bot force-pushed the renovate/major-stryker-monorepo branch from 8d086a8 to 47efb24 Compare December 25, 2020 13:15
@renovate renovate bot force-pushed the renovate/major-stryker-monorepo branch from 47efb24 to 62d8485 Compare January 19, 2021 05:00
to match changes in stryker.
Tried jest-runner but couldn't get it working (doesn't find tests in sandbox).
Running jest with some more arguments to speed up execution.

https://stryker-mutator.io/blog/announcing-stryker-4-mutation-switching
https://stryker-mutator.io/docs/stryker/configuration/
@karfau karfau requested a review from brodybits January 19, 2021 08:06
@karfau
Copy link
Member

karfau commented Jan 19, 2021

@brodybits I added 7b5073c to make the update work.
Still took 50 minutes to build the report on my machine, but the scores look very similar to the existing ones.

Please have a look when your time allows.

stryker.conf.json Outdated Show resolved Hide resolved
@@ -38,8 +38,7 @@
},
"dependencies": {},
"devDependencies": {
"@stryker-mutator/core": "^3.3.1",
"@stryker-mutator/javascript-mutator": "^3.3.1",
"@stryker-mutator/core": "^4.3.1",
Copy link
Member

Choose a reason for hiding this comment

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

FYI I think this is slightly outdated now. If we merge this PR, Renovate should be able to make the update for us.

@brodybits
Copy link
Member

LGTM in general. I did try this on the cloud with 8 vCPU, finished in about 51 minutes. (I got timeouts when running Stryker from master for some reason.)

This was referenced Mar 13, 2021
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.

None yet

3 participants