green-coding-solutions / eco-ci-energy-estimation

Eco CI Energy estimation for Github Actions Runner VMs
MIT License
48 stars 10 forks source link

PR-Comment fails on Github Enterprise #48

Closed dan-mm closed 5 months ago

dan-mm commented 7 months ago
jreimone commented 6 months ago

I cannot second that. I see PR comments in our Github Enterprise.

ArneTR commented 6 months ago

@jreimone Thanks for the input @dan-mm Can you link the source why this ticket was opened from our side so we can narrow it down

dan-mm commented 5 months ago

@ArneTR - Unfortunately I can't link it - this was from the conference workshop, one of the attendees mentioned it to me in person, and I wrote the ticket in that moment. Specifically he mentioned that they were hosting their own github API (possible with enterprise), so he needed the url that we call to be configurable. Its possible that in @jreimone didn't have this issue if they were still using github's standard api.

The work to support this has already done here: https://github.com/green-coding-berlin/eco-ci-energy-estimation/commit/12c5a5a332fc2d1cb605acd00759fde7c2c87c24

jreimone commented 5 months ago

@dan-mm is correct, we use the standard API.

ArneTR commented 5 months ago

closed for now. Please reopen if problem occurs again for anyone.