Skip to content

Added experimental watch mode - #175

Merged
lolleko merged 6 commits into
masterfrom
feature/watch-mode
Aug 20, 2018
Merged

Added experimental watch mode#175
lolleko merged 6 commits into
masterfrom
feature/watch-mode

Conversation

@lolleko

@lolleko lolleko commented Aug 19, 2018

Copy link
Copy Markdown
Member

Added support for tsc flags -w/--watch

The watch mode is currently not really incremental and will emit the whole project if a file is changed.
See #174

@lolleko
lolleko requested a review from Perryvw August 19, 2018 14:11
Comment thread src/Compiler.ts Outdated
);
}

const errorDiagnostic: ts.Diagnostic = {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I would prefer you create this and immediately fill in the correct values in afterProgramCreate since it's not used anywhere else.

Comment thread src/Compiler.ts Outdated
if (options.outDir !== options.rootDir) {
const relativeSourcePath = path.resolve(sourceFile.fileName)
.replace(path.resolve(rootDir), "");
.replace(path.resolve(rootDir), "");

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

You sure this should be gone? I like the old version better.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

auto format /shrug

@lolleko
lolleko merged commit 688fe72 into master Aug 20, 2018
@Perryvw
Perryvw deleted the feature/watch-mode branch August 21, 2018 06:45
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.

2 participants