Skip to content
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

Fix problems when running this template on case-sensitive operating systems #1

Closed
wants to merge 1 commit into from

Conversation

@ferrybig
Copy link

ferrybig commented Oct 31, 2019

At the moment, when running this template on case sensitive platforms (Linux & Mac OS), the application fails to run, as certain files cannot be found, this fixes this.

Error that gets thrown when running the template without modifications:

npm run dev

> nodejs-api-typescript-starter@1.0.0 dev /home/fernando/Downloads/nodejs-api-typescript-starter
> nodemon --exec ts-node ./src/index.ts

[nodemon] 1.19.4
[nodemon] reading config ./nodemon.json
[nodemon] to restart at any time, enter `rs`
[nodemon] or send SIGHUP to 4249 to restart
[nodemon] ignoring: ./.git/**/* node_modules/**/node_modules
[nodemon] watching dir(s): src/**/*
[nodemon] watching extensions: js,json,ts
[nodemon] starting `ts-node ./src/index.ts`
[nodemon] spawning
[nodemon] child pid: 4269
[nodemon] watching 20 files

/home/fernando/Downloads/nodejs-api-typescript-starter/node_modules/ts-node/src/index.ts:245
    return new TSError(diagnosticText, diagnosticCodes)
           ^
TSError: ⨯ Unable to compile TypeScript:
src/config/index.ts:1:31 - error TS2307: Cannot find module '../common/baseConfiguration'.

1 import BaseConfiguration from "../common/baseConfiguration";
                                ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

    at createTSError (/home/fernando/Downloads/nodejs-api-typescript-starter/node_modules/ts-node/src/index.ts:245:12)
    at reportTSError (/home/fernando/Downloads/nodejs-api-typescript-starter/node_modules/ts-node/src/index.ts:249:19)
    at getOutput (/home/fernando/Downloads/nodejs-api-typescript-starter/node_modules/ts-node/src/index.ts:357:34)
    at Object.compile (/home/fernando/Downloads/nodejs-api-typescript-starter/node_modules/ts-node/src/index.ts:415:32)
    at Module.m._compile (/home/fernando/Downloads/nodejs-api-typescript-starter/node_modules/ts-node/src/index.ts:493:43)
    at Module._extensions..js (internal/modules/cjs/loader.js:827:10)
    at Object.require.extensions.(anonymous function) [as .ts] (/home/fernando/Downloads/nodejs-api-typescript-starter/node_modules/ts-node/src/index.ts:496:12)
    at Module.load (internal/modules/cjs/loader.js:685:32)
    at Function.Module._load (internal/modules/cjs/loader.js:620:12)
    at Module.require (internal/modules/cjs/loader.js:723:19)
[nodemon] app crashed - waiting for file changes before starting...

@escalonc

This comment has been minimized.

Copy link
Owner

escalonc commented Oct 31, 2019

Hey man it looks cool, thanks for your help.
But I saw your PR after a I sent a couple of commits, could you help me renaming all the files in the common folder with camel case style?

@escalonc

This comment has been minimized.

Copy link
Owner

escalonc commented Oct 31, 2019

I already add a couple of changes in the CI to prevent this.

@escalonc escalonc closed this Oct 31, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.