Skip to content

Conversation

pat
Copy link
Contributor

@pat pat commented Apr 28, 2017

Simple enough (and it installed fine for me with this change using 2.4.1). Wonder if having the upper limit on Ruby versions is actually helpful? Though I'm guessing those who've been involved in this project for some time may know better :)

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.5%) to 94.058% when pulling 6479b68 on pat:patch-1 into 99552d4 on aws:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.09%) to 94.439% when pulling f4e6160 on pat:patch-1 into 99552d4 on aws:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 94.567% when pulling 90d6e9a on pat:patch-1 into 91b4466 on aws:master.

@asaf-erlich
Copy link
Contributor

I've merged in your code into our internal repo (you can see where the commit is mentioned above). The PR was out of date so I couldn't just merge in your commit. Thank you very much for the contribution. It should have been out in the agent release that occurred in December.

@pat
Copy link
Contributor Author

pat commented Jan 24, 2018

Thanks! And now that MRI 2.5 is out, would a PR for that be welcome as well? :)

@asaf-erlich
Copy link
Contributor

Of course. :)

@analytically
Copy link

@asaf-erlich Would you be able to merge the 2.5 support, this is critical to install the agent on Ubuntu 18.04 LTS, thanks

@asaf-erlich
Copy link
Contributor

@rohkat-aws is probably the right person to ping for this since he still works on the team.

Good luck,
-Asaf

philstrong pushed a commit to philstrong/aws-codedeploy-agent that referenced this pull request May 21, 2021
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.

4 participants