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

Add docker support to help building dist files #237

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

Conversation

mstred
Copy link

@mstred mstred commented May 25, 2018

Hi @gregjacobs... although you're the one supposed to be building the project's dist files, I think it would be kinda helpful to use an easier step to achieve that, either for yourself or even for particular forks of Autolinker.

In this PR, I'm adding a Dockerfile which uses a base image with node and ruby (due to jsduck), a .dockerignore and a docker-compose.yml file with an example of how to use it.

Feel free to add any suggestions or even improvements on this.

@gregjacobs
Copy link
Owner

Hey @mstred, thanks for the PR. I was just thinking: JSDuck was a great documentation generator, but unfortunately it seems abandoned at this point and doesn't support TypeScript (of which I'd like to update Autolinker to). Would it be helpful for me to just remove the JSDuck dependency?

@mstred
Copy link
Author

mstred commented May 31, 2018

That was the only reason I looked for some way to put both node and ruby in the same base image, as it's required for JSDuck to be installed as a ruby gem. Removing it from the actual stack would make ir even leaner.
However, when trying to run npm install over a pure node-based docker image, gulp-jsduck post-install hook will look for a jsduck gem executable and it'll fail, as it won´t be able to find it in an environment without rubygems.

Here's an snippet of what happens in this situation:

$ npm i
npm WARN deprecated [email protected]: gulp-util is deprecated - replace it, following the guidelines at https://medium.com/gulpjs/gulp-util-ca3b1f9f9ac5
npm WARN deprecated [email protected]: gulp-util is deprecated - replace it, following the guidelines at https://medium.com/gulpjs/gulp-util-ca3b1f9f9ac5
npm WARN deprecated [email protected]: Use the built-in module in node 9.0.0 or newer, instead
npm WARN deprecated [email protected]: connect 2.x series is deprecated
npm WARN deprecated [email protected]: please upgrade to graceful-fs 4 for compatibility with current and future versions of Node.js
npm WARN deprecated [email protected]: Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated [email protected]: Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated [email protected]: please upgrade to graceful-fs 4 for compatibility with current and future versions of Node.js

> [email protected] install /srv/Autolinker.js/node_modules/jsduck
> npm run-script configure


> [email protected] configure /srv/Autolinker.js/node_modules/jsduck
> node scripts/configure.js

Configuring JSDuck
Could not find jsduck. Attempting to install.
Could not find gem executable.
Please manually install JSDuck.

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

Successfully merging this pull request may close these issues.

2 participants