Code review comment for lp:~leonardr/launchpadlib/multiversion

Revision history for this message
Leonard Richardson (leonardr) wrote :

This branch changes launchpadlib to take advantage of lazr.restful's 'version' constructor argument. Instead of a service root of http://api.launchpad.net/beta/, you now use a service root of http://api.launchpad.net/ and a version of "beta". This makes it much easier to specify which version of a web service you want to use.

This branch also changes the default version of the web service to 'devel'. Here is the plan (not yet ratified by distro developers):

1. We get a launchpadlib release based on this branch into Lucid before the feature freeze. At this point anyone who writes launchpadlib scripts with beta versions of Lucid will be using the unstable 'devel' version of the web service.

2. Before Lucid is released, we make some changes to the Launchpad web service and freeze a '1.0' release.

3. We make a small launchpadlib release that does nothing but change the default web service version to '1.0'. Anyone who writes launchpadlib scripts with the released
version of Lucid will be using the frozen '1.0' version of the web service.

This is flacoste's plan; I would prefer to change the default web service version to '1.0' in this branch, and not mess with Lucid any more after this.

Either way, by releasing a '1.0' version of the web service and tying it to Lucid, we will be able to retire the 'beta' version when we retire Karmic. If we don't get this done, we won't be able to retire the 'beta' version until we retire Lucid.

« Back to merge proposal