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(package): restructure package internals #2714
Changes from all commits
dfe8dc2
648ea68
8f7f4a0
4be037b
be3ca7d
8d989e1
86de21a
0043c21
b255e5a
2ce5228
62ea482
90910da
8c9becf
d6e4ee9
74e68de
b3f1f2a
34d29da
0399af4
53d2a84
90dd4b0
766c414
4c6a9d9
34c3640
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,4 @@ | ||
{ | ||
"require": ["test/setup.js"], | ||
"require": ["dist/test/setup.js"], | ||
"timeout": 10000 | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,13 +1,14 @@ | ||
**/* | ||
!*.d.ts | ||
!bin/** | ||
!src/** | ||
!schema/** | ||
!*.js | ||
src/**/*.map | ||
src/**/*.ts | ||
!{src,src-generated}/**/*.d.ts | ||
!schema/*.json | ||
!readme.md | ||
!LICENSE | ||
!CHANGELOG.md | ||
stryker.conf.js | ||
test | ||
dist/test | ||
/schema | ||
/reports | ||
/testResources | ||
/.nyc_output | ||
.vscode | ||
.nycrc.json | ||
*.tsbuildinfo | ||
tsconfig?(.*).json | ||
.mocharc.jsonc | ||
.nyc_output | ||
.gitattributes |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
export * from './dist/src/check'; |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
module.exports = require("./dist/src/check") | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Should JS and .D.TS files be on github tho? 🤔 There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Great question! I would actually like to have it as a *.ts file, but that would mean we would need to compile those files in place (not to the When we drop support for node < 12.7 we can start using I don't want to make an exception for some files to be compiled in place, because that also means making an exception in our clean script and making the output files hidden in vscode config (i don't like hidden files 😢). This all adds complexity that I don't want. Do you agree with this? In short: we should be able to remove these files once we can use |
This file was deleted.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
export * from './dist/src/core'; |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
module.exports = require("./dist/src/core") |
This file was deleted.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
export * from './dist/src/logging'; |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
module.exports = require("./dist/src/logging") |
This file was deleted.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
export * from './dist/src/plugin'; |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
module.exports = require("./dist/src/plugin") |
This file was deleted.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
export * from './dist/src/report'; |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
module.exports = require("./dist/src/report") |
This file was deleted.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,3 @@ | ||
export * from './checker'; | ||
export * from './check-result'; | ||
export * from './check-status'; |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,11 @@ | ||
export { File } from './file'; | ||
export { Position } from './position'; | ||
export { Location } from './location'; | ||
export { Range } from './range'; | ||
export { Mutant } from './mutant'; | ||
export * from '../../src-generated/stryker-core'; | ||
export * from './report-types'; | ||
export * from './stryker-options-schema'; | ||
export * from './partial-stryker-options'; | ||
export * from './instrument'; | ||
export * from './mutant-coverage'; |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,11 +1,9 @@ | ||
import Position from './position'; | ||
import { Position } from './position'; | ||
|
||
/** | ||
* A location in the source code which can span multiple lines and/or columns. | ||
*/ | ||
interface Location { | ||
export interface Location { | ||
start: Position; | ||
end: Position; | ||
} | ||
|
||
export default Location; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think thats an issue
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Importing
it
from mocha is optional. Mocha will make sureit
is globally available. I've removed it since we're never importingit
.