Skip to content
This repository has been archived by the owner on Nov 16, 2023. It is now read-only.

Fixes issue #10 #11

Closed
wants to merge 2 commits into from
Closed

Fixes issue #10 #11

wants to merge 2 commits into from

Conversation

raerpo
Copy link

@raerpo raerpo commented Jun 10, 2017

This PR solves a few little typos and change some devdependencies definitions in the README

@msftclas
Copy link

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

bowdenk7 pushed a commit that referenced this pull request Jul 20, 2017
bowdenk7 pushed a commit that referenced this pull request Jul 20, 2017
port changes from PR #11 and update package.lock from latest npm version
@bowdenk7
Copy link
Contributor

Ported your changes to #30 thanks!!

@bowdenk7 bowdenk7 closed this Jul 20, 2017
cobola added a commit to cobola/TypeScript-Node-Starter that referenced this pull request Dec 29, 2017
* fix typo

* fix: copying issues

* fix: fix for tslint

* port changes from PR microsoft#11

* Added survey links that I forgot ><

* fix: fix lint warnings for test files

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

* updating package-lock

* looks like new version of npm adds a blank line after each npm install. Committing it

* updated specific .d.ts files to point to definitely typed file. Also added recommended extensions using vs code's extensions.json

* update to node 8 on travis

* removed yarn as npm 5 makes it mostly unneccesary and it's causing some issues with CI

* add attach debug configuration

- Attach debug configuration
- Scripts: debug, serve-debug, watch-debug
- Readme

* fix: word

* .

* Update build task

* change tests to not always pass

* separate server binding to port from other app config

this seems a bit crude but allows running the tests and having them take
care of setting up the port to use

* might as well also add the change for issue microsoft#41

* Update README.md

Added missing "node_modules/*" path mapping to README.

* feat(*): update several dependencies

* Removed extra "s" in sourceMap compilerOptions

* Update README.md

Added missing "node_modules/*" path mapping to README.

* change tests to not always pass

* feat(*): update several dependencies

* Removed extra "s" in sourceMap compilerOptions

* separate server binding to port from other app config

this seems a bit crude but allows running the tests and having them take
care of setting up the port to use

* fix up tests

* update repo field

* made some changes to mongo connection
@cobola cobola mentioned this pull request Apr 19, 2018
brittanydrandolph pushed a commit that referenced this pull request Jun 17, 2022
brittanydrandolph pushed a commit that referenced this pull request Jun 17, 2022
port changes from PR #11 and update package.lock from latest npm version
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants