search

OpenFeign / feign

Feign makes writing java http clients easier
Apache License 2.0
9.52k stars 1.93k forks source link

Enable Build Scans on develocity.commonhaus.dev #2543

Closed bdemers closed 2 months ago

bdemers commented 2 months ago

cc: @velo

[!NOTE] This repo (or org) will need a secret named DEVELOCITY_ACCESS_KEY (This was recently done for Morphia, the process should be the same here.)

bdemers commented 2 months ago

I'm still getting back up to speed here. I'm assuming this is part of the projects enrollment in the Commonhaus Foundation and this is one of the benefits, considering Develocity is a pay service?

@kdavisk6, correct. Its a way to get some observability/metrics into your build. What takes the longest, track flaky tests, compare builds to see what changed between them (dependencies, java version, etc). There is also other functionality like build caching, and ways to speed up your tests, but... collecting metrics is always the first steps (measure before optimization).

kdavisk6 commented 2 months ago

Great. I'll approve the PR, but wait to test the workflows until we get the Secret configured.

bdemers commented 2 months ago

Looks like there was a YAML formatting error: https://github.com/OpenFeign/feign/actions/runs/10776636436

Fixed in: 41f3cc90a284dac84a2b6cc21fb2a64181b365f5