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

chore(deps): update @stryker-mutator/core -> 5 - devdependencies #235

Merged
merged 1 commit into from Jun 9, 2021

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented May 14, 2021

WhiteSource Renovate

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
@stryker-mutator/core (source) ^4.5.1 -> ^5.0.1 age adoption passing confidence

Release Notes

stryker-mutator/stryker-js

v5.0.1

Compare Source

Bug Fixes

v5.0.0

Compare Source

Bug Fixes
  • sandbox: make directory if not exists before symlinking node_modules (#​2856) (40f9a1d)
  • vue-tsx: support parsing tsx script in .vue file (#​2850) (dc66c28)
Features
BREAKING CHANGES
  • range: The range property is no longer present on a mutant. Note, this is a breaking change for plugin creators only.
  • options: "perTest" is now the default value for "coverageAnalysis" when the configured test runner is not "command". Explicitly set "coverageAnalysis": "off" manually to opt-out of this behavior.
  • node: Node 10 is no longer supported. Please use Node 12 or higher.
  • serialize: Having a non-JSON-serializable value in your configuration won't be sent to the child process anymore. If you really need them in your test runner configuration, you should isolate those values and put them in test runner-specific config files, loaded by the test runner plugin itself, for example, jest.config.js, karma.conf.js, webpack.config.js.
  • ignore patterns: Stryker will no longer use a git command to determine which files belong to your project. Instead, it will rely on sane defaults. You can change this behavior by defining ignorePatterns.
  • ignore patterns: The files configuration option is deprecated and will be removed in a future release. Please use ignorePatterns instead.

This:

{
  "files": ["foo.js"]
}

Is equivalent to:

{
  "ignorePatterns": ["**", "!foo.js"]
}
  • reporter api: Changes to Reporter and TestRunner plugin API of Stryker
  • jest-runner: Support for project type create-react-app-ts is dropped from the jest-runner.

v4.6.0

Compare Source

Features

4.5.1 (2021-03-11)

Bug Fixes
  • babel-transformer: respect top of the file comments/pragma (#​2783) (ca42276)
  • instrumenter: corect mutant location in .vue and .html files (547a25c), closes #​2790
  • peer-deps: use correct peer dep version (a6ca0f2)

Configuration

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

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

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


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

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

@brodybits
Copy link
Member

I think we should test this before merging, would also like it if we could switch to the Jest runner as I started working on in PR #208.

@renovate renovate bot force-pushed the renovate/major-stryker-monorepo branch from d6d5748 to c000d87 Compare May 28, 2021 10:15
@renovate renovate bot changed the title Update @stryker-mutator/core -> 5 - devDependencies chore(deps): update @stryker-mutator/core -> 5 - devdependencies Jun 6, 2021
@renovate renovate bot force-pushed the renovate/major-stryker-monorepo branch 2 times, most recently from 8be06bb to 0848720 Compare June 8, 2021 18:45
@renovate renovate bot force-pushed the renovate/major-stryker-monorepo branch from 0848720 to 8d0baf6 Compare June 8, 2021 18:47
@karfau
Copy link
Member

karfau commented Jun 8, 2021

Just started it on my machine and it works in it's current state.
I think we should rather update #208 to take care of switching to jest runner, so I would say we land this one.

@brodybits
Copy link
Member

I think we should rather update #208 to take care of switching to jest runner, so I would say we land this one.

I guess you mean land this one first and then update the changes for the Jest runner (from PR #208) to work with Stryker JS 5. This is the approach that I would take.

I would be a bit curious, though, if we can measure the difference in performance between Stryker 4.5.1 and 5.0.1. I did this before with 2 vCPU cores, which seemed to be the most consistent with GitHub.

I do find it pretty unfortunate that the Stryker run takes so long on GitHub.

@karfau karfau merged commit ae9ba07 into master Jun 9, 2021
@karfau karfau deleted the renovate/major-stryker-monorepo branch June 9, 2021 20:05
karfau pushed a commit to ChALkeR/xmldom that referenced this pull request Jul 27, 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