Code review comment for ~jugmac00/lazr.restful:apply-inclusive-naming

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

This technically changes a few bits of public API (`lazr.restful.metazcml.webservice_sanity_checks` and `lazr.restful.testing.webservice.Dummy*URL`). In practice I think this is fine and doesn't need a major version bump or anything - lazr.restful only has one known significant user (Launchpad itself) and that doesn't use any of the relevant names - but just in case I'd be inclined to at least make a note of it in NEWS and bump the version to 1.2.0.

review: Approve

« Back to merge proposal