Skip to content
This repository was archived by the owner on Sep 21, 2022. It is now read-only.

Fix: require default config file by absolute path#442

Merged
j0tunn merged 2 commits intomasterfrom
fix/default.config
Apr 20, 2016
Merged

Fix: require default config file by absolute path#442
j0tunn merged 2 commits intomasterfrom
fix/default.config

Conversation

@j0tunn
Copy link
Contributor

@j0tunn j0tunn commented Apr 20, 2016

If no config file specified gemini will try to read some default configs. In case of js|json files gemini will just require them. For now it requires them by name and obviously fails.

@levonet levonet added the review label Apr 20, 2016
@j0tunn
Copy link
Contributor Author

j0tunn commented Apr 20, 2016

@tormozz48 @sipayRT

@sipayRT
Copy link
Member

sipayRT commented Apr 20, 2016

ok

@j0tunn j0tunn closed this Apr 20, 2016
@j0tunn j0tunn deleted the fix/default.config branch April 20, 2016 12:51
@levonet levonet removed the review label Apr 20, 2016
@coveralls
Copy link

Coverage Status

Coverage increased (+2.0%) to 78.799% when pulling 88de4a9 on fix/default.config into 240d057 on master.

@j0tunn j0tunn restored the fix/default.config branch April 20, 2016 13:01
@j0tunn j0tunn reopened this Apr 20, 2016
@levonet levonet added the review label Apr 20, 2016
@j0tunn j0tunn merged commit 70b3ede into master Apr 20, 2016
@j0tunn j0tunn deleted the fix/default.config branch April 20, 2016 13:01
@levonet levonet removed the review label Apr 20, 2016
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.

4 participants