Skip to content

chore: oppgrader actions/labeler fra v5 til v6#348

Open
AnjaAalerud wants to merge 2 commits into
masterfrom
chore/oppgrader-labeler-v6
Open

chore: oppgrader actions/labeler fra v5 til v6#348
AnjaAalerud wants to merge 2 commits into
masterfrom
chore/oppgrader-labeler-v6

Conversation

@AnjaAalerud

Copy link
Copy Markdown
Contributor
  • Oppdaterer labeler.yml-syntaks: fjerner any-glob-to-any-file (v5) og bruker direkte glob-liste (v6)

- Oppdaterer labeler.yml-syntaks: fjerner any-glob-to-any-file (v5)
  og bruker direkte glob-liste (v6)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@AnjaAalerud AnjaAalerud requested a review from a team as a code owner June 23, 2026 07:53
@AnjaAalerud

Copy link
Copy Markdown
Contributor Author

I følge Claude så kan vi ikke oppgradere til versjon 6 av labeler uten å oppdatere konfig syntaks.

@jolarsen jolarsen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Vi skal ikke bruke v5/v6 men SHA - slik det er i fp-gha-workflows.

Men: Hvorfor har vi gjeninnført labeler i fp-inntektsmelding?
Vi fjernet dem nylig fra alle repos.

@AnjaAalerud

Copy link
Copy Markdown
Contributor Author

Vi skal ikke bruke v5/v6 men SHA - slik det er i fp-gha-workflows.

Men: Hvorfor har vi gjeninnført labeler i fp-inntektsmelding? Vi fjernet dem nylig fra alle repos.

Dette ble lagt til nylig for å kun få med kontraktsendringer under whats changes når vi releaser nye kontrakter. Jeg fikk hjelp av claude. Kan jeg løse det uten å bruke labeler?

@jolarsen

Copy link
Copy Markdown
Contributor

@AnjaAalerud det viktige her er når release-drafter kjøres (fra build-kontrakt.yml).
Ser ikke denne grei ut ? https://github.com/navikt/fp-inntektsmelding/releases - hva er for mye her?

Vi hadde tidligere et oppsett i andre repos som satte label kontrakt hvis endringen var i kontrakter. Så hadde bygg/draft-stegene et filter på label kontrakt. Overdrevet og ikke forstått eller vedlikeholdt. Vi fjernet labeler fra de siste 4-5 reppene tidlig mai 2026.
Mye bedre å ha en build.yml som ikke aktiveres av endring i kontrakt og så en build-kontrakt som bare trigges av endringer i kontrakt (build-kontrakt tar med release-draft).

@sonarqubecloud

Copy link
Copy Markdown

@AnjaAalerud

Copy link
Copy Markdown
Contributor Author

@AnjaAalerud det viktige her er når release-drafter kjøres (fra build-kontrakt.yml). Ser ikke denne grei ut ? https://github.com/navikt/fp-inntektsmelding/releases - hva er for mye her?

Vi hadde tidligere et oppsett i andre repos som satte label kontrakt hvis endringen var i kontrakter. Så hadde bygg/draft-stegene et filter på label kontrakt. Overdrevet og ikke forstått eller vedlikeholdt. Vi fjernet labeler fra de siste 4-5 reppene tidlig mai 2026. Mye bedre å ha en build.yml som ikke aktiveres av endring i kontrakt og så en build-kontrakt som bare trigges av endringer i kontrakt (build-kontrakt tar med release-draft).

Den er fin, vi merget dette med labeler tidligere så hvis du ser på versjon 0.1.6 feks så får man med mer enn det man ønsker. De andre har vi manuelt fjernet det som ikke var relevant.

@jolarsen

Copy link
Copy Markdown
Contributor

Den er fin, vi merget dette med labeler tidligere så hvis du ser på versjon 0.1.6 feks så får man med mer enn det man ønsker. De andre har vi manuelt fjernet det som ikke var relevant.

0.1.6 har med seg en dependency-oppgradering av kontrakt. Det er sjelden, men nyttig info som bør være med. Det er vanlig kotyme på releaser av andre/eksterne bibliotek. Har med kompatibilitet å gjøre.
Så lenge det ikke kommer med commits utenom kontrakt så blir vurderingen rundt det å stikke seg ut fra applikasjonsporteføljen.
Uansett action med versjon vN er ut. Ratchet og SHA er in. Ref fp-gha-workflows

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.

3 participants