Skip to content
This repository has been archived by the owner on Feb 2, 2018. It is now read-only.

chore: dev setup for custom editors/IDEs #8

Merged
merged 1 commit into from
Oct 3, 2017

Conversation

bajtos
Copy link
Contributor

@bajtos bajtos commented Oct 2, 2017

  • Rename "vscode-test" task to "test:dev".

  • Add a new task "build:watch" for running tsc in watch mode.

  • Update DEVELOPING.md with instructions for people not using VS Code.

See #7 (comment) and #2.

@bajtos bajtos mentioned this pull request Oct 2, 2017
@bajtos bajtos self-assigned this Oct 2, 2017
DEVELOPING.md Outdated

### Other editors/IDEs

1. Start the build process in background via `npm run tsc-watch`. It will run TypeScript compiler in backround, watching and recompiling files as you change them.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Should be npm run build:watch. It's not running as a background task for me (not that I mind it. Like it in the foreground so I can see errors).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good catch. I'll rephrase the sentence to make this more clear.

virkt25 added a commit that referenced this pull request Oct 3, 2017
Will remove `package.json` once #8 has landed.
 - Rename "vscode-test" task to "test:dev".

 - Add a new task "build:watch" for running tsc in watch mode.

 - Update DEVELOPING.md with instructions for people not using VS Code.
@bajtos bajtos force-pushed the chore/vscode-less-workflow branch from 94bc1a9 to 00adb15 Compare October 3, 2017 11:41
@bajtos bajtos requested a review from raymondfeng as a code owner October 3, 2017 11:41
@bajtos bajtos merged commit cfa8183 into master Oct 3, 2017
@bajtos bajtos deleted the chore/vscode-less-workflow branch October 3, 2017 11:52
virkt25 added a commit that referenced this pull request Oct 4, 2017
Will remove `package.json` once #8 has landed.
virkt25 added a commit that referenced this pull request Oct 5, 2017
Will remove `package.json` once #8 has landed.
virkt25 added a commit that referenced this pull request Oct 11, 2017
Will remove `package.json` once #8 has landed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants