Skip to content

Conversation

@coadler
Copy link
Contributor

@coadler coadler commented Jun 29, 2021

No description provided.

Copy link

@jawnsy jawnsy left a comment

Choose a reason for hiding this comment

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

good catch, thanks for fixing!

@jawnsy
Copy link

jawnsy commented Jul 11, 2021

@coadler Can you rebase, remove the -e from the Makefile (it doesn't seem necessary on Ubuntu), and merge this? You can also close this and I can take it over if you'd like

https://github.com/cdr/enterprise-helm/blob/043db4cfc140ec880c9afecb144a40deb7408eea/Makefile#L41

@coadler coadler force-pushed the colin/ci-exit-unfmtd branch from 0c505bb to f982948 Compare December 8, 2021 23:06
@coadler coadler force-pushed the colin/ci-exit-unfmtd branch from 669407c to 9452b76 Compare December 8, 2021 23:22
@coadler coadler merged commit b2d4210 into main Dec 8, 2021
@coadler coadler deleted the colin/ci-exit-unfmtd branch December 8, 2021 23:28
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.

2 participants