Skip to content

build(Makefile): add lint-code target#549

Merged
bepri merged 1 commit into
mainfrom
work/lint-code
Jun 5, 2026
Merged

build(Makefile): add lint-code target#549
bepri merged 1 commit into
mainfrom
work/lint-code

Conversation

@mr-cal
Copy link
Copy Markdown
Collaborator

@mr-cal mr-cal commented Jun 5, 2026

Adds a new make lint-code target.


  • I've followed the contribution guidelines.
  • I've signed the CLA.
  • I've successfully run make lint && make test.
  • I've added or updated any relevant documentation.
  • In documents I changed, I added a meta description if one was missing.
  • I've updated the relevant release notes.

Signed-off-by: Callahan Kovacs <callahan.kovacs@canonical.com>
Copilot AI review requested due to automatic review settings June 5, 2026 11:30
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds a dedicated make lint-code target to run code-focused linters, and updates the existing make lint target to compose lint-code with the non-code lint targets (docs, packaging, lockfile, and actions).

Changes:

  • Refactors lint to depend on a new aggregated lint-code target plus existing non-code lint steps.
  • Adds lint-code as a convenience target for running only code-related linters.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mr-cal mr-cal marked this pull request as ready for review June 5, 2026 11:41
@mr-cal mr-cal requested a review from a team June 5, 2026 11:41
@tigarmo tigarmo requested a review from a team June 5, 2026 11:48
@bepri bepri merged commit 875e9e7 into main Jun 5, 2026
13 checks passed
@bepri bepri deleted the work/lint-code branch June 5, 2026 16:01
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