Skip to content
This repository has been archived by the owner on Oct 14, 2020. It is now read-only.

No More Root #77

Merged
merged 13 commits into from
Aug 14, 2014
Merged

No More Root #77

merged 13 commits into from
Aug 14, 2014

Conversation

xcthulhu
Copy link
Contributor

Closes #29

This was referenced Aug 13, 2014
Closed
Closed
@jhoffner
Copy link
Member

Looks good. The only thing that I wonder if we should change is how we order the dockerfile. Maybe we should install all core language dependencies at the beginning and wait to install any packages/gems until the end, after all of the languages have been installed.

@xcthulhu
Copy link
Contributor Author

Probably makes sense.

One thing I’m uncertain about: made codewarrior’s login shell /usr/sbin/nologin, this might screw stuff up.

Writing mongodb tests now, it doesn’t need root to be started…

On Aug 14, 2014, at 2:00 PM, Jake Hoffner notifications@github.com wrote:

Looks good. The only thing that I wonder if we should change is how we order the dockerfile. Maybe we should install all core language dependencies at the beginning and wait to install any packages/gems until the end, after all of the languages have been installed.

—
Reply to this email directly or view it on GitHub.

@xcthulhu xcthulhu mentioned this pull request Aug 14, 2014
Closed
@xcthulhu
Copy link
Contributor Author

Okay, I made mongodb tests, this is gtg

jhoffner added a commit that referenced this pull request Aug 14, 2014
@jhoffner
Merge pull request #77 from xcthulhu/no-more-root
62e727c 
No More Root
@jhoffner jhoffner merged commit 62e727c into codewars:master Aug 14, 2014
@xcthulhu xcthulhu mentioned this pull request Aug 14, 2014
Closed
jhoffner added a commit that referenced this pull request Mar 5, 2015
@jhoffner
Merge pull request #77 from xcthulhu/no-more-root
048f4af 
No More Root
jhoffner added a commit that referenced this pull request Mar 5, 2015
@jhoffner
Merge pull request #77 from xcthulhu/no-more-root
7faf321 
No More Root
jhoffner added a commit that referenced this pull request Mar 5, 2015
@jhoffner
Merge pull request #77 from xcthulhu/no-more-root
3860b59 
No More Root
jhoffner added a commit that referenced this pull request Mar 5, 2015
@jhoffner
Merge pull request #77 from xcthulhu/no-more-root
76c996f 
No More Root
jhoffner added a commit that referenced this pull request Mar 5, 2015
@jhoffner
Merge pull request #77 from xcthulhu/no-more-root
0e81fa5 
No More Root
jhoffner added a commit that referenced this pull request Mar 5, 2015
@jhoffner
Merge pull request #77 from xcthulhu/no-more-root
c91c548 
No More Root
jhoffner added a commit that referenced this pull request Mar 5, 2015
@jhoffner
Merge pull request #77 from xcthulhu/no-more-root
f2417a9 
No More Root
@xcthulhu
Copy link
Contributor Author

xcthulhu commented Mar 7, 2015

Hi Codewars/codewars-runner,
A message from Matthew Wampler-Doty was left on IQ Elite for you.
Click here a.iqelite.com/j/0JmRlo/esb/?hg=fL8o3NduV5 to read your message

The IQ Elite Team

This message was meant for reply@reply.github.com. If you do not want to continue receiving such mails from IQ Elite, please click on this link: a.iqelite.com/common/unsubscribe/?code=65bbe887&email=reply@reply.github.com&hg=fL8o3NduV5

IQ Elite is a service with main office: C/ Gran De Gràcia 15, 1a 1a, 08012 Barcelona, Spain.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Don't be Root
2 participants
@xcthulhu @jhoffner