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

Adding Rubygem for Ruby Applications #21

Conversation

christopherhein
Copy link
Contributor

@christopherhein christopherhein commented Dec 7, 2017

Why:

  • Currently tested to run a sinatra application
  • Uses the same mp-compiler to build out the specs
  • Uses an alpine image out of the box to run

Todo:

  • need to set sane defaults in the gem to allow you to run without ports defined
  • build out sharding tutorial

Resolves #8

@christopherhein christopherhein force-pushed the feature/add-ruby-mechanisms branch 2 times, most recently from 394938d to 536bc0e Compare December 8, 2017 20:21
@@ -0,0 +1,74 @@
# Contributor Covenant Code of Conduct
Copy link
Contributor

Choose a reason for hiding this comment

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

I'd like to add a code of conduct to the overall repository, and I'd like to use the Microsoft CoC:

https://opensource.microsoft.com/codeofconduct/

Let's do this as a separate PR?

@@ -0,0 +1,21 @@
The MIT License (MIT)
Copy link
Contributor

Choose a reason for hiding this comment

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

I'd like to have a single LICENSE at the top, rather than a lower-level one in a language implemenation. Hope that makes sense...


## Contributing

Bug reports and pull requests are welcome on GitHub at https://github.com/christopherhein/metaparticle. This project is intended to be a safe, welcoming space for collaboration, and contributors are expected to adhere to the [Contributor Covenant](http://contributor-covenant.org) code of conduct.
Copy link
Contributor

Choose a reason for hiding this comment

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

Change to point to the https://github.com/metaparticle-io/package repo.

Delete the code of conduct part given we'll add it in a separate PR for the whole repo.


## Code of Conduct

Everyone interacting in the Metaparticle project’s codebases, issue trackers, chat rooms and mailing lists is expected to follow the [code of conduct](https://github.com/christopherhein/metaparticle/blob/master/CODE_OF_CONDUCT.md).
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's delete and add these as part of the Code of Conduct PR (or wait to merge this, and redirect it)

@brendandburns
Copy link
Contributor

@christopherhein thanks for the PR, I had a few comments...

@christopherhein christopherhein force-pushed the feature/add-ruby-mechanisms branch 2 times, most recently from feeb9a0 to 13ec2c9 Compare December 13, 2017 18:22
@christopherhein
Copy link
Contributor Author

@brendandburns addressed the current feedback, thank you!

**Why:**

* Currently tested to run a `sinatra` application
* Uses the same `mp-compiler` to build out the specs
* Uses an `alpine` image out of the box to run
@brendandburns
Copy link
Contributor

LGTM thanks!

@brendandburns brendandburns merged commit 2f4695e into metaparticle-io:master Dec 15, 2017
@christopherhein christopherhein deleted the feature/add-ruby-mechanisms branch January 12, 2018 18:29
cgmcintyr pushed a commit to cgmcintyr/package that referenced this pull request Feb 27, 2018
**Why:**

* Currently tested to run a `sinatra` application
* Uses the same `mp-compiler` to build out the specs
* Uses an `alpine` image out of the box to run
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.

None yet

2 participants