Merge ~cjwatson/turnip/+git/turnipcake:woke into ~canonical-launchpad-branches/turnip/+git/turnipcake:master

Proposed by Colin Watson
Status: Merged
Approved by: Colin Watson
Approved revision: 324d2baec3d8b4441673de3b542e26749b424e1f
Merge reported by: Otto Co-Pilot
Merged at revision: not available
Proposed branch: ~cjwatson/turnip/+git/turnipcake:woke
Merge into: ~canonical-launchpad-branches/turnip/+git/turnipcake:master
Diff against target: 249 lines (+237/-0)
2 files modified
.pre-commit-config.yaml (+16/-0)
.woke.yaml (+221/-0)
Reviewer Review Type Date Requested Status
Jürgen Gmach Approve
Review via email: mp+452355@code.staging.launchpad.net

Commit message

Check inclusive naming via the woke pre-commit hook

To post a comment you must log in.
Revision history for this message
Jürgen Gmach (jugmac00) wrote :

Would it make sense to add a link to https://git.launchpad.net/lp-inclusive-naming/tree/.woke.yaml which seems to be the canonical source for the yaml configuration? Either in a comment or at least in the commit message?

review: Approve
Revision history for this message
Colin Watson (cjwatson) wrote :

I was trying to avoid changing the actual file since that would show up in diff output, but I've amended the commit message.

There was an error fetching revisions from git servers. Please try again in a few minutes. If the problem persists, contact Launchpad support.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches