Better branch name error

degica / barcelona

PaaS built on top of AWS

MIT License

52 stars 6 forks source link

Better branch name error #723

Closed davidsiaw closed 2 years ago

davidsiaw commented 2 years ago

Right now if you deploy a reviewapp with the branch name containing big letters it will fail. This PR will make the message better so the user can fix the problem without having to inspect logs and trace the error back to the source where the verification occurs.

degicat commented 2 years ago

@pmq20 can you help us review this PR, please?

Metallion commented 2 years ago

Thank you! ^_^