Prepare 1.0.0 release

[simondel]

With the 1.0.0 release with Stryker we will make the following changes:

Preparation

• Readonly options #365
• Remove side effects from stryker plugins #667
• Release v1.0.0 of all depedencies that we maintain ourselves #799 (release v1 of dependencies)
• Potential Memory Leak #920 (support big projects)
• Rename 'project' config key to 'projectType'
• Rename 'timeoutMs' config key to 'timeoutMS'

Execution

• Remove deprecated Factories from stryker-api.
• Remove ES5 mutator
• Set 'javascript' as the default mutator
• Remove deprecated reporter config key
• Remove deprecated project config key
• Remove deprecated 'default' projectType for jest (in favor of 'custom')
• Remove deprecated timeoutMs config key
• Rename packages:
  • stryker => @stryker-mutator/core
  • stryker-xxx-yyy => @stryker-mutator/xxx-yyy
  • stryker-jasmine => @stryker-mutator/jasmine-framework
  • stryker-typescript => @stryker-mutator/typescript
• Remove config key from being injectable
• Remove getLogger and setImplementation from stryker-api/logging
• Remove all deprecated apis/messages/etc
• Update plugin loader to load plugins from @stryker-mutator (instead of stryker-)
• Enable lerna versioning mode (not independent, other one)
• Update stryker-cli to install @stryker-mutator/core instead of stryker and stryker-api
• Update package.json keywords to @stryker-mutator/
• Update keyword plugin to 'stryker-mutator-plugin'

Clean up

• Deprecate Stryker packages: https://docs.npmjs.com/cli/deprecate (let's wait ~ a week for this)
• Write awesome blog article
• Update handbook https://github.com/stryker-mutator/stryker-handbook/tree/v1-release
• Update website quickstart: https://github.com/stryker-mutator/stryker-mutator.github.io/tree/v1-release

This original post will be kept up-to-date.

[nicojs]

Why would we no longer support mutators as strings? mutator: 'typescript' could be a shorthand for

mutator: {
   type: 'typescript',
   excludedMutations: []
}

[simondel]

I posted that here: #652 (comment)

EDIT: I have removed

• Remove support for mutators as strings. Only objects will be supported

because that's not the way we will do things as discussed in #652

[simondel]

What about renaming the reporter property of the stryker config to reporters? It's an array after all. We could already implement this with a deprecation warning for people still using the property reporter and remove the reporter property in 1.0.0

[nicojs]

Sounds good. Let's create a separate issue for that

[simondel]

I created #793 and I have updated the main post of this issue

[nicojs]

I've created #799 and updated the main post.

[bartekleon]

@simondel, @nicojs I am trying to solve some easy issues and refactors you need. I am quite newbie in your project, try to understand how it really works, and to understand TS in 100%. Do you have some instructions / tips, how can I work out / find out myself in that big project? You can see some progress in fork

[simondel]

Awesome :) We have a mono-repo. All the stryker plugins that we maintain ourselves are in here. We use Lerna to bind it all together and ensure the packages work together locally. If you run commands such as npm install from the root of the repo, they will be executed in each package.

Perhaps a nice one to start with is this: #1076
It's isolated to two packages which are a lot smaller in size than the main codebase of Stryker 👍 You could either fix both packages in one pull request or split it up into two pull requests. Whatever works for you.

I would also suggest reading the contribution guide if you haven't read it already.

Unfortunately we can't make all the changes for 1.0.0. I'll split the original post up between preparation and execution.

[bartekleon]

@simondel i don't know if you checked what I have done yesterday in fork. I could make pull request today but i have solved #860 and part of the #654 i think. Also did some minor changes to mutators

[bartekleon]

also i have question about packages/stryker-karma-runner/package.json. In all files you define project [projectType] to be default and then you change it to custom. Is it issue or I missed some line in config

[bartekleon]

I have tried to test but it crashes while testing with one test ->

1) A babel project with the `only` option enabled
       should be able to transpile the input files:
     Error: Timeout of 2000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (C:\Users\19gro\Desktop\stryker\packages\stryker-babel-transpiler\test\integration\BabelProjects.it.js)

after next 3 npm run test all testes passed 😛 - it's strange

[simondel]

Since we released it, let's close this :)

[nicojs]

Are you sure it was well prepared? ;)