Closed collinsnji closed 6 years ago
Looks neat ! @collinsnji
What I'm doubtful about is if we need the package-lock.json
to be added to the commits ?
I think that's a configuration file that's generated while building/running
What are your views on this @sneakycr0w ?
You shouldn’t. The dependencies will be in package.json.
Zachary E Sohovich Full Stack Web Developer https://sneakycr0w.com zach@sneakycr0w.com
On Oct 2, 2017, at 10:54 PM, Anshuman Verma notifications@github.com wrote:
Looks neat ! @collinsnji What I'm doubtful about is if we need the package-lock.json to be added to the commits ? I think that's a configuration file that's generated while building/running What are your views on this @sneakycr0w ?
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.
Right, exactly.
Apparently in this PR you have included package-lock
. Please add another commit to remove it @collinsnji 😄
Also i think every other thing is in place, what say @sneakycr0w
Oh right. I did not noticed when I committed the package-lock
file. I'll remove it
Thanks ! Actually i should've added that to .gitignore earlier 😅
Haha yeah. One more thing, how will the webapp be hosted? Heroku?
I don't have any such preference, what are your preferences on this ?
I think Heroku will be best for this :) @sneakycr0w what do you think?
Right, heroku sounds good to me. Let's hear from @sneakycr0w
Fixes #4
By submitting this pull request I confirm I've read and complied with the below declarations.
Added/Fixed {Feature/Bug}
, notUpdate README.md
orAdded new code
.