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

Changed require configuration to path relative to current working dir… #427

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

BigBadAlien
Copy link

…ectory

For example working directory is /home/foo/project_with_gemini/gemini/.
We use js or json default path configuration or use -c option.
Run ./gemini or ./gemini -c .very_special_configuration.json.
And we get configuration relative to current working directory:
/home/foo/project_with_gemini/gemini/.gemini.conf.json
/home/foo/project_with_gemini/gemini/.gemini.conf.js
/home/foo/project_with_gemini/gemini/.very_special_configuration.json
not in directory anywhere in the node_modules.
Fixed #418.

@levonet levonet added the review label Apr 12, 2016
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.9%) to 77.225% when pulling 9879fe3 on BigBadAlien:js_json_configuration_relative_path into d11c52c on gemini-testing:master.

@BigBadAlien BigBadAlien force-pushed the js_json_configuration_relative_path branch from 9879fe3 to 8e6bf55 Compare April 12, 2016 10:43
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.9%) to 77.225% when pulling 8e6bf55 on BigBadAlien:js_json_configuration_relative_path into d11c52c on gemini-testing:master.

…ectory

For example working directory is `/home/foo/project_with_gemini/gemini/`.
We use js or json default path configuration or use `-c` option.
Run `./gemini` or `./gemini -c .very_special_configuration.json`.
And we get configuration relative to current working directory:
/home/foo/project_with_gemini/gemini/.gemini.conf.json
/home/foo/project_with_gemini/gemini/.gemini.conf.js
/home/foo/project_with_gemini/gemini/.very_special_configuration.json
not in directory anywhere in the node_modules.
Fixed gemini-testing#418.
@BigBadAlien BigBadAlien force-pushed the js_json_configuration_relative_path branch from 8e6bf55 to a4fd607 Compare April 12, 2016 11:55
@BigBadAlien BigBadAlien reopened this Apr 12, 2016
@levonet levonet added review and removed review labels Apr 12, 2016
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.8%) to 77.273% when pulling a4fd607 on BigBadAlien:js_json_configuration_relative_path into d11c52c on gemini-testing:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.8%) to 77.273% when pulling a4fd607 on BigBadAlien:js_json_configuration_relative_path into d11c52c on gemini-testing:master.

@BigBadAlien
Copy link
Author

Functional and unit tests in summary covers all added code.
Decrease of coverage looks like some wrong logic in counting of coverage.
Testing cover for browser of course will be lower, because added not browser code and its not needed testing but still counted in covering

@SwinX
Copy link
Contributor

SwinX commented May 13, 2016

/cc @Cody0

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants