start-dev launch option (ts-node-dev) #234

GrayStrider · 2019-10-06T12:28:05Z

allows to work on project without re-building each time

Update references from bowdenk7 repo to Microsoft repo

added package-lock for npm5 and fixed bug with flash messages

Added `build` `preLaunchTask` for debug configuration

Removed duplication in npm scripts `serve` and `watch`

…MaxOptions'. fix err TypeScript-Node-Starter/src/controllers/api.ts[6, 9]: missing whitespace, 8 files fix bugs of issue 15

fit bugs of issue 15 and some other bugs caused by tslint

port changes from PR microsoft#11 and update package.lock from latest npm version

fix: copying issues

fix typo

…lash-types Simplify augmented module for express-flash

…-nodejs-versions Specify NodeJS version as strings in Travis configuration

📝 remove reference to legacy file. Closes microsoft#190

This commit rewrites linting support in the project to be based solely on the ESLint as advised by the TSLint tool authors: https://medium.com/palantir/tslint-in-2019-1a144c2317a9 The migration is based on default, recommended settings for TypeScript in ESLint and is expected to be updated in future to better fit project goals. All references has been updated and replaced with relevant ESLint context: - dependencies migration from TSLint to ESLint - VSCode configuration changes to support ESLint exension - VSCode extensions recommendation changes - `.eslintrc` and `.eslintignore` configuration files added - all error level problems in the source files are covered by this migration Thanks!

✨ Migrate TSLint to ESLint. Closes microsoft#209

fix lint error ``` PS> npm run lint > [email protected] lint C:\Users\JipingWang\source\repos\api > tsc --noEmit && eslint '*/**/*.{js,ts}' --quiet --fix Oops! Something went wrong! :( ESLint: 5.16.0. No files matching the pattern "'*/**/*.{js,ts}'" were found. Please check for typing mistakes in the pattern. npm ERR! code ELIFECYCLE npm ERR! errno 2 npm ERR! [email protected] lint: `tsc --noEmit && eslint '*/**/*.{js,ts}' --quiet --fix` npm ERR! Exit status 2 npm ERR! npm ERR! Failed at the [email protected] lint script. npm ERR! This is probably not a problem with npm. There is likely additional logging output above. npm ERR! A complete log of this run can be found in: npm ERR! C:\Users\JipingWang\AppData\Roaming\npm-cache\_logs\2019-07-18T15_26_05_580Z-debug.log ```

fix lint error

Fix lint warnings

Bumps [eslint-utils](https://github.com/mysticatea/eslint-utils) from 1.4.0 to 1.4.2. - [Release notes](https://github.com/mysticatea/eslint-utils/releases) - [Commits](mysticatea/eslint-utils@v1.4.0...v1.4.2) Signed-off-by: dependabot[bot] <[email protected]>

Fix typo

…rn/eslint-utils-1.4.2 Bump eslint-utils from 1.4.0 to 1.4.2

See: https://mongoosejs.com/docs/deprecations.html#ensureindex Thanks!

Use update indexes access in Mongoose configuration. Fixes microsoft#225

@CarlosSolrac

- upgrade to latest packages - cast `req.user` as `UserDocument` - update logger to use latest Winston syntax - disable `no-inferrable-types`, Mongoose use unified topology Thx to @CarlosSolrac

peterblazejewicz

If accepted, it would require solution for serving static assets from public directory. At the moment, content usually pre-built like css (from scss) could not be used when started in that mode.
For example such minimal change does the trick, but is just a pointer on how to achieve this, could be better one:

add cross-env as dev dependency

"start-dev": "cross-env NODE_ENV=development dotenv -- nodemon -e ts -x ts-node --files src/server.ts"

app.ts:

const staticAssetsRoot = (process.env.NODE_ENV === "development") ?
    path.join(__dirname, "../dist") :
    __dirname;

app.use(
    express.static(path.join(staticAssetsRoot, "public"), { maxAge: 31557600000 })
);

package.json

GrayStrider · 2019-10-10T03:26:21Z

If accepted, it would require solution for serving static assets from public directory. At the moment, content usually pre-built like css (from scss) could not be used when started in that mode.

@peterblazejewicz
I just launched it and everything seems to work fine? No unresolved resources or anything, css seems to be built as well.

perhaps, --files option takes care of things? Could someone test it and see what exactly doesn't work, if anything?

GrayStrider · 2019-10-10T03:49:22Z

Nevermind, found a much better option, one sec

GrayStrider · 2019-10-10T04:16:50Z

package.json

@@ -10,7 +10,7 @@
 "license": "MIT",
 "scripts": {
 "start": "npm run serve",
- "start-dev": "dotenv -- nodemon -e ts -x ts-node --files src/server.ts",
+ "start-dev": "dotenv -- ts-node-dev --watch views src/server.ts",


takes a few seconds to reload after changes, unlike nodemon;
--watch views will also trigger reload on any change in views folder (still need to refresh the page to see them)
unfortunatelly, couldn't make it trigger refresh on scss changes in public folder (--watch public,views), works when putting scss in views folder though.

GrayStrider · 2019-12-21T01:36:49Z

Are there any other adjustments needs to be made?

anthony-bernardo · 2020-11-27T15:40:14Z

why this is not merged ? this is a must have when you are working on a project

Bowden Kelly and others added 30 commits May 9, 2017 13:28

initial commit

d7905d7

README Title Update

9a6ccfb

update README

502ac88

removed yarn section

a9664be

added nodemon as dev dependency

c2d938a

Update references from bowdenk7 repo to Microsoft repo

9c3e0df

Update README.md

f943c5e

Update README.md

888d8a2

Merge branch 'master' into master

3df842e

Merge pull request microsoft#2 from julienmartin/master

8c35e5d

Update references from bowdenk7 repo to Microsoft repo

added package-lock for npm5 and fixed bug with flash messages

2eae2fd

Merge pull request microsoft#6 from Microsoft/bugFix

c9b6765

added package-lock for npm5 and fixed bug with flash messages

Update README.md

4608459

Added build preLaunchTask for debug configuration

aff88f2

Merge pull request microsoft#7 from Deilan/vscode-task-build

54c291b

Added `build` `preLaunchTask` for debug configuration

Removed duplication in npm scripts serve and watch

d423251

Removed duplicate README section

a7ef8d5

fix typo

0cec294

Merge pull request microsoft#8 from Deilan/npm-run-server-dup

61d4512

Removed duplication in npm scripts `serve` and `watch`

fix: copying issues

d6d880c

fix: fix for tslint

5472454

fit error TS2559: Type '4' has no properties in common with type 'Min…

76c4593

…MaxOptions'. fix err TypeScript-Node-Starter/src/controllers/api.ts[6, 9]: missing whitespace, 8 files fix bugs of issue 15

Merge pull request microsoft#26 from cobola/master

bb5281e

fit bugs of issue 15 and some other bugs caused by tslint

port changes from PR microsoft#11

5c796ca

Merge pull request microsoft#30 from Microsoft/testing

9601c65

port changes from PR microsoft#11 and update package.lock from latest npm version

Merge pull request microsoft#24 from kevguy/fix-copy

6d9fee1

fix: copying issues

Merge pull request microsoft#22 from backmeupplz/patch-1

38be103

fix typo

Added survey links that I forgot ><

a50b2a9

fix: fix lint warnings for test files

c80e236

Bump tasks.json version from 0.10 to 2.00. Add default build task.

da5b277

Orta and others added 16 commits July 15, 2019 17:08

Merge pull request microsoft#206 from peterblazejewicz/feat/express-f…

a2e090f

…lash-types Simplify augmented module for express-flash

Merge pull request microsoft#204 from peterblazejewicz/fix/travis-yml…

db9e1ba

…-nodejs-versions Specify NodeJS version as strings in Travis configuration

Merge pull request microsoft#205 from peterblazejewicz/fix/190

4a287b6

📝 remove reference to legacy file. Closes microsoft#190

Merge pull request microsoft#211 from peterblazejewicz/feat/209

9817bef

✨ Migrate TSLint to ESLint. Closes microsoft#209

Merge pull request microsoft#212 from JipingWang/patch-2

606ccf0

fix lint error

Fix lint warnings

c489bde

Merge pull request microsoft#214 from mathiasvr/lint-fix

1bf5836

Fix lint warnings

Fix typo

4479a64

Merge pull request microsoft#223 from cbenz/patch-1

40e7ba9

Fix typo

Merge pull request microsoft#224 from microsoft/dependabot/npm_and_ya…

722ebf8

…rn/eslint-utils-1.4.2 Bump eslint-utils from 1.4.0 to 1.4.2

Use update indexes access in Mongoose configuration. Fixes microsoft#225

8592e13

See: https://mongoosejs.com/docs/deprecations.html#ensureindex Thanks!

Merge pull request microsoft#226 from peterblazejewicz/fix/225

ca14e7c

Use update indexes access in Mongoose configuration. Fixes microsoft#225

Update packages, add tests, add type casts (microsoft#229)

1409fd7

- upgrade to latest packages - cast `req.user` as `UserDocument` - update logger to use latest Winston syntax - disable `no-inferrable-types`, Mongoose use unified topology Thx to @CarlosSolrac

GrayStrider force-pushed the patch-nodemon branch 2 times, most recently from 5a58703 to 2299dc1 Compare October 6, 2019 13:39

peterblazejewicz reviewed Oct 6, 2019

View reviewed changes

package.json Outdated Show resolved Hide resolved

start-dev launch option (nodemon, node-ts)

9746234

GrayStrider force-pushed the patch-nodemon branch from 2299dc1 to 9746234 Compare October 6, 2019 15:55

GrayStrider changed the title ~~start-dev launch option (nodemon, node-ts)~~ start-dev launch option (nodemon, ts-node) Oct 10, 2019

switch to ts-node-dev, watch views, remove dotenv-cli

97ac1a3

GrayStrider commented Oct 10, 2019

View reviewed changes

GrayStrider changed the title ~~start-dev launch option (nodemon, ts-node)~~ start-dev launch option (ts-node-dev) Oct 10, 2019

brittanydrandolph force-pushed the master branch from fbc1328 to 1e15e8b Compare June 24, 2022 15:41

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

start-dev launch option (ts-node-dev) #234

start-dev launch option (ts-node-dev) #234

GrayStrider commented Oct 6, 2019

peterblazejewicz left a comment •

edited

GrayStrider commented Oct 10, 2019 •

edited

GrayStrider commented Oct 10, 2019

GrayStrider Oct 10, 2019 •

edited

GrayStrider commented Dec 21, 2019

anthony-bernardo commented Nov 27, 2020

start-dev launch option (ts-node-dev) #234

Are you sure you want to change the base?

start-dev launch option (ts-node-dev) #234

Conversation

GrayStrider commented Oct 6, 2019

peterblazejewicz left a comment • edited

Choose a reason for hiding this comment

GrayStrider commented Oct 10, 2019 • edited

GrayStrider commented Oct 10, 2019

GrayStrider Oct 10, 2019 • edited

Choose a reason for hiding this comment

GrayStrider commented Dec 21, 2019

anthony-bernardo commented Nov 27, 2020

peterblazejewicz left a comment •

edited

GrayStrider commented Oct 10, 2019 •

edited

GrayStrider Oct 10, 2019 •

edited