Skip to content
This repository has been archived by the owner on Jun 1, 2022. It is now read-only.

Feature: return proper branch reference and commit id with events #5

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

cscheib
Copy link

@cscheib cscheib commented Dec 7, 2019

This PR addresses the "Known Limitation" that previously only returned ref: master and commit: HEAD

This seemed to require an undue amount of effort to build as-is, so I took the liberty of updating the Makefile and Dockerfile to look more like brigade-github-app gateway's.

I've also updated the documentation accordingly.

I left these commits pretty separate in case someone wants to cherry-pick them.

@cscheib cscheib mentioned this pull request Dec 7, 2019
…ields that I care about, update server.go to utilize these fields/values

Signed-off-by: Chris Scheib <chris@scheib.io>
change image back to alpine to fix an issue with /tmp file writes
update Dockerfile to put binary back in original location

Signed-off-by: Chris Scheib <chris@scheib.io>
Signed-off-by: Chris Scheib <chris@scheib.io>
Signed-off-by: Chris Scheib <chris@scheib.io>
Signed-off-by: Chris Scheib <chris@scheib.io>
Signed-off-by: Chris Scheib <chris@scheib.io>
Signed-off-by: Chris Scheib <chris@scheib.io>
Signed-off-by: Chris Scheib <chris@scheib.io>
@radu-matei
Copy link
Contributor

This is excellent work, thank you, @cscheib!
I'll create a Brigade instance and test this next week.

We should also look into updating the name of the repo / gateway to keep up with Azure Pipelines name changing..

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants