Skip to content

fix(ci): raport postepow nie mogl odczytac ruchu, ktory ma raportowac - #177

Merged
msgwing merged 1 commit into
mainfrom
report-traffic-perms
Aug 17, 2026
Merged

msgwing merged 1 commit into
mainfrom
report-traffic-perms

Conversation

@msgwing

@msgwing msgwing commented Aug 17, 2026

Copy link
Copy Markdown
Owner

Pierwszy bieg poszedł bez najważniejszych liczb: endpointy ruchu odmówiły tokenowi Actions — Resource not accessible by integration. Mapują się na uprawnienie administration: read, którego workflow nie prosił.

Raport bez liczby odwiedzin jest jedyną rzeczą, którą ten raport być nie może. GitHub trzyma ruch 14 dni i nikt inny go nie zapisuje, więc pominięty bieg to dziura w jedynym trendzie, jaki ten projekt ma.

Dodatkowo porażka jest teraz core.error, a nie core.warning, a tabela mówi, że API odmówiło, a nie że nie odpowiedziało. Raport, który po cichu gubi liczby, dla których istnieje, czyta się jak spokojny tydzień.

The first run went out with its headline numbers missing: the traffic
endpoints refused the Actions token with 'Resource not accessible by
integration'. They map to the administration:read permission, which the
workflow did not request.

A report without visitor counts is the one thing this report cannot be.
GitHub keeps traffic for 14 days and nobody else is writing it down, so a
missed run is a hole in the only trend the project has.

The failure is now core.error rather than core.warning, and the table says
the API refused rather than that it did not respond. A report that quietly
drops the numbers it exists to carry reads like a calm week.
@msgwing
msgwing enabled auto-merge August 17, 2026 21:22
@github-actions github-actions Bot added the ci label Aug 17, 2026
@msgwing
msgwing merged commit 9c8d9a3 into main Aug 17, 2026
34 of 35 checks passed
@msgwing
msgwing deleted the report-traffic-perms branch August 17, 2026 21:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant