Robert Collins wrote:
> This is probably due to lp taking too long to mark the branch as
> merged: if it takes longer than pqm takes to try again, then pqm will
> see it as still pending. Possibly we should:
> mark things we succeed at as approved
> adding a comment that it landed ok
>
> or mark it as merged.
>
> I'm a little worried about triggering launchpadlib errors though,
> because lp is going to be updating the status at the same time - we
> can collide.
I think it would be reasonable to have your script notice that "Nothing
to be Merged" obviously means that the branch is already merged...
John
=:->
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (Cygwin)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Robert Collins wrote:
> This is probably due to lp taking too long to mark the branch as
> merged: if it takes longer than pqm takes to try again, then pqm will
> see it as still pending. Possibly we should:
> mark things we succeed at as approved
> adding a comment that it landed ok
>
> or mark it as merged.
>
> I'm a little worried about triggering launchpadlib errors though,
> because lp is going to be updating the status at the same time - we
> can collide.
I think it would be reasonable to have your script notice that "Nothing
to be Merged" obviously means that the branch is already merged...
John
=:->
-----BEGIN PGP SIGNATURE----- enigmail. mozdev. org/
Y+TsACgkQJdeBCY SNAAPIMQCgqjpPM mQSCREDGp9/ 5OdQKQVN YQqYSNCR5lVb6+ 31+
Version: GnuPG v1.4.9 (Cygwin)
Comment: Using GnuPG with Mozilla - http://
iEYEARECAAYFAkv
V8YAn34mPZMKIzB
=hVtp
-----END PGP SIGNATURE-----