Skip to content

ci: add bitbake-lint workflow for pull requests#49

Merged
Ricardo Salveti (ricardosalveti) merged 1 commit into
qualcomm-linux:mainfrom
igoropaniuk:ci/bitbake-lint
Jul 2, 2026
Merged

ci: add bitbake-lint workflow for pull requests#49
Ricardo Salveti (ricardosalveti) merged 1 commit into
qualcomm-linux:mainfrom
igoropaniuk:ci/bitbake-lint

Conversation

@igoropaniuk

Copy link
Copy Markdown
Contributor

Add a Bitbake Lint GitHub Actions workflow using
qualcomm-linux/bitbake-lint-action to catch recipe/metadata issues on pull requests.

Fine-tuning:

  • paths filter: only run on bitbake metadata changes (*.bb, *.bbappend, *.bbclass, *.inc, *.conf) to skip doc/CI-only PRs
  • diff_only: lint only files touched in the PR diff
  • release: infer the Yocto release from LAYERSERIES_COMPAT_qcom in conf/layer.conf
  • hide info: drop info-severity noise
  • exit_zero: advisory rollout, findings annotate but do not block merge
  • permissions: pull-requests/checks write for inline diff annotations

Add a Bitbake Lint GitHub Actions workflow using
qualcomm-linux/bitbake-lint-action to catch recipe/metadata issues
on pull requests.

Fine-tuning:
- paths filter: only run on bitbake metadata changes
  (*.bb, *.bbappend, *.bbclass, *.inc, *.conf) to skip doc/CI-only PRs
- diff_only: lint only files touched in the PR diff
- release:  infer the Yocto release from LAYERSERIES_COMPAT_qcom
  in conf/layer.conf
- hide info: drop info-severity noise
- exit_zero: advisory rollout, findings annotate but do not block merge
- permissions: pull-requests/checks write for inline diff annotations

Signed-off-by: Igor Opaniuk <igor.opaniuk@oss.qualcomm.com>

@ricardosalveti Ricardo Salveti (ricardosalveti) left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Will approve so we can merge and tune as we see the need.

@koenkooi Koen Kooi (koenkooi) left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Let's see what this catches!

@quaresmajose

Copy link
Copy Markdown
Contributor

I left another comment on qualcomm-linux/meta-qcom#2668 (comment)

@ricardosalveti

Copy link
Copy Markdown
Collaborator

I left another comment on qualcomm-linux/meta-qcom#2668 (comment)

Based on that thread I believe the current logic is fine, will merge so we can continue the validation.

@ricardosalveti Ricardo Salveti (ricardosalveti) merged commit 0263951 into qualcomm-linux:main Jul 2, 2026
17 of 18 checks passed
@quaresmajose

Copy link
Copy Markdown
Contributor

I left another comment on qualcomm-linux/meta-qcom#2668 (comment)

Based on that thread I believe the current logic is fine, will merge so we can continue the validation.

right

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.

4 participants