#pr-reviews - June 2020

13 messages

Pull Request Reviews for Cloud Posse Projects

Mikhail Naletov
Mikhail Naletovover 6 years ago
Adam Crews
Adam Crewsover 6 years ago
Good morning,
Could I get a review on https://github.com/cloudposse/terraform-aws-s3-bucket/pull/27 when someone has a moment? This replaces pr#23 in the same repo with an alternate/more flexible implementation.
Adam Crews
Adam Crewsover 6 years ago
I created 2 more PR’s for the above project that close out a couple of tickets. I would love a review of those also. Thank you!
Erik Osterman (Cloud Posse)
Erik Osterman (Cloud Posse)over 6 years ago
so sorry everyone - @Andriy Knysh (Cloud Posse) has been out of town this week
PePe Amengual
PePe Amengualover 6 years ago
np
Erik Osterman (Cloud Posse)
Erik Osterman (Cloud Posse)over 6 years ago(edited)
you're most patient @PePe Amengual 😃
PePe Amengual
PePe Amengualover 6 years ago
slow and steady you are bound to succeed
Tyrone Meijn
Tyrone Meijnover 6 years ago
https://github.com/cloudposse/terraform-aws-vpc/pulls could some PRs be reviewed for this module? These are open since march 26th.
Erik Osterman (Cloud Posse)
Erik Osterman (Cloud Posse)over 6 years ago
We are working on increasing our collaborators for Pr reviews
Erik Osterman (Cloud Posse)
Erik Osterman (Cloud Posse)over 6 years ago
We have started a group with about 7 now. Still writing up the documentation they need to get started. Sorry we have over 270 open PRs
joey
joeyover 6 years ago(edited)
i was going to start using https://github.com/cloudposse/slack-notifier but i want buttons https://api.slack.com/legacy/message-buttons .. i wonder if you could do something like?
SLACK_BUTTON_COUNT=2
SLACK_BUTTON_1_TEXT=approve
SLACK_BUTTON_1_URL=<https://codefresh/${CF_BUILD_ID}/approve>
SLACK_BUTOTN_2_TEXT=deny
SLACK_BUTTON_2_URL=<https://codefresh/${CF_BUILD_ID}/deny>

but erik suggested i might just be better off using gomplate and i think he's right
Frank
Frankover 6 years ago
Good morning! Could someone please take a look at my PR? https://github.com/cloudposse/terraform-aws-ses/pull/2
Thanks!