search

navapbc / template-infra

A template to set up foundational infrastructure for your application in AWS
Apache License 2.0
9 stars 2 forks source link

Don't comment on PR after destroying PR environment #674

Closed lorenyu closed 1 week ago

lorenyu commented 1 week ago

Ticket

n/a

Changes

see title

Context for reviewers

I think the PR environment destroyed comment is also noisy since it sends an email notification and it's never actionable.

Testing

Developed and tested in platform-test in https://github.com/navapbc/platform-test/pull/120

lorenyu commented 1 week ago

Sorry for another PR related to this, wanted to refine just a little bit more

rocketnova commented 1 week ago

Sorry for another PR related to this, wanted to refine just a little bit more

No worries at all. Happy that you're making this functionality so nice.

lorenyu commented 1 week ago

Oops I thought i merged in that change before I merged the PR. I'm just gonna push it to main directly.