Skip to content
This repository has been archived by the owner on Aug 30, 2021. It is now read-only.

Adding ssl support back into 0.4.0 #548

Merged
merged 1 commit into from
May 28, 2015

Conversation

jloveland
Copy link
Contributor

adding ssl back in, moving scripts to script folder so we can ./script/gereratate-ssl-certs.sh

@@ -2,19 +2,36 @@

// Load the module dependencies
var config = require('../config'),
path = require('path'),
path = require('path'),
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

indentation is off.

@ilanbiala ilanbiala added this to the 0.4.0 milestone May 6, 2015
@ilanbiala ilanbiala self-assigned this May 6, 2015
@ilanbiala
Copy link
Member

@jloveland why script and not scripts? @lirantal any issue with merging this in?

@jloveland jloveland force-pushed the generate-sslcerts branch from 6c994e9 to 022ee83 Compare May 7, 2015 06:14
@jloveland
Copy link
Contributor Author

@ilanbiala, this is just a preference..most projects I work on put all the scripts into a script folder, mostly inspired by holman's dotfiles. However, I can change script back to scripts if you want. let me know.

@ilanbiala
Copy link
Member

@jloveland honestly I think it just comes down to it rolling off the tongue easier when telling other people and it seeming more natural because it's a folder of scripts.

@jloveland jloveland force-pushed the generate-sslcerts branch from 022ee83 to 3e1d0b5 Compare May 8, 2015 14:10
@jloveland
Copy link
Contributor Author

@ilanbiala I changed script back to scripts

@ilanbiala
Copy link
Member

@lirantal LGTY?

@lirantal
Copy link
Member

looks ok

ilanbiala added a commit that referenced this pull request May 28, 2015
@ilanbiala ilanbiala merged commit 3ea400f into meanjs:0.4.0 May 28, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants