Skip to content

CI: Add local exceptions to flatpak-builder-lint#13200

Merged
RytoEX merged 1 commit intoobsproject:masterfrom
tytan652:fb-lint-exceptions
Mar 8, 2026
Merged

CI: Add local exceptions to flatpak-builder-lint#13200
RytoEX merged 1 commit intoobsproject:masterfrom
tytan652:fb-lint-exceptions

Conversation

@tytan652
Copy link
Collaborator

@tytan652 tytan652 commented Mar 8, 2026

Description

For some reason the action can no longer check if the URL generated in the metainfo is reachable.
So to avoid keeping CI failing, local exceptions rule are added.

Unfortunately it requires to keep a copy of exceptions already submitted to Flathub.

Motivation and Context

Keep the linter functional over just cutting it off until we find a better long-term solution.

There is a suspicion that GitHub runner have a rate limits of HTTP request to GitHub…

How Has This Been Tested?

CI on the PR on my fork and this PR's CI.

Types of changes

  • CI

Checklist:

  • My code has been run through clang-format.
  • I have read the contributing document.
  • My code is not on the master branch.
  • The code has been tested.
  • All commit messages are properly formatted and commits squashed where appropriate.
  • I have included updates to all appropriate documentation.

For some reason the action can no longer check if the URL generated in
the metainfo is reachable.
So to avoid keeping CI failing, local exceptions rule are
added.

Unfortunately it requires to keep a copy of exceptions already submitted
to Flathub.
@tytan652 tytan652 force-pushed the fb-lint-exceptions branch from 08c7ded to 4d1579f Compare March 8, 2026 12:23
@tytan652 tytan652 changed the title CI: Add local excpetions to flathub-builder-lint CI: Add local exceptions to flatpak-builder-lint Mar 8, 2026
Copy link
Member

@RytoEX RytoEX left a comment

Choose a reason for hiding this comment

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

This is fine for now, though it would be nice to understand why this is happening and if there is a better solution.

@RytoEX RytoEX added this to the OBS Studio 32.1 milestone Mar 8, 2026
@RytoEX RytoEX merged commit 60533db into obsproject:master Mar 8, 2026
15 checks passed
@tytan652 tytan652 deleted the fb-lint-exceptions branch March 8, 2026 16:12
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.

2 participants