Skip to content

(feature) Support node module imports #39

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.

Sign up for GitHub

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

Closed
wants to merge 6 commits into from
Closed

(feature) Support node module imports #39

Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
eb1cd6f
Support absolute paths and NODE_PATH
vinhlh Oct 4, 2018
14d3c40
Make test number fancy
vinhlh Oct 4, 2018
9d0538a
Add support for pulling from node modules if relative path does not e…
audiolion Oct 23, 2018
13d3a8c
lint
audiolion Oct 27, 2018
12b9df6
prettier
audiolion Oct 27, 2018
d0471bd
Merge branch 'master' into feature/support-node-module-imports
evenchange4 Jan 24, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 13 additions & 2 deletions src/macro.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,8 +10,19 @@ import compileWithFragment from './utils/compileWithFragment';
// console.log(printAST(referencePath.parentPath))

const cwd = fs.realpathSync(process.cwd());
const resolvePathFromCwd = relativePath =>
path.resolve(cwd, process.env.NODE_PATH || '.', relativePath);
const resolvePathFromCwd = relativePath => {
const resolvedPath = path.resolve(
cwd,
process.env.NODE_PATH || '.',
relativePath,
);
return fs.exists(resolvedPath, exists => {
if (exists) {
return resolvedPath;
}
return path.resolve(cwd, 'node_modules', relativePath);
});
};

function graphqlMacro({
references,
Expand Down