Skip to content

feat(lint)!: don't lint the donation key#6094

Open
bepri wants to merge 2 commits intomainfrom
work/dont-lint-donation
Open

feat(lint)!: don't lint the donation key#6094
bepri wants to merge 2 commits intomainfrom
work/dont-lint-donation

Conversation

@bepri
Copy link
Member

@bepri bepri commented Mar 12, 2026

After discussion with @mr-cal and @lengau, it was decided that it wasn't very helpful to enforce the donation key on a project. It is commonly ignored and entirely unwanted for proprietary snaps like Spotify, Discord, or Firefox.

This will break project builds that are currently ignoring this key.


  • 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.
  • I've updated the relevant release notes.

@bepri bepri self-assigned this Mar 12, 2026
@bepri bepri marked this pull request as ready for review March 12, 2026 14:34
@bepri bepri requested review from a team and mr-cal as code owners March 12, 2026 14:34
@bepri bepri requested a review from lengau March 12, 2026 14:34
Co-authored-by: Michael DuBelko <michael.dubelko@canonical.com>
Signed-off-by: Imani Pelton <imani.pelton@canonical.com>
Copy link
Collaborator

@mr-cal mr-cal left a comment

Choose a reason for hiding this comment

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

What do you think about silently ignoring donation in lint: ignore: metadata:?

I realize this may make the scope of work larger than a fly-by PR, but we don't have a deprecation warning in Snapcraft 8 that this key is going to be dropped. Ideally, someone who heeds deprecation warnings in Snapcraft 8 won't get surprised with a broken build in Snapcraft 9.

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