Skip to content
This repository was archived by the owner on Mar 2, 2026. It is now read-only.

fix: normalize / test deprecation of 'Client.reserve_ids'#103

Merged
tseaver merged 5 commits intomasterfrom
101-reserve_ids-deprecation-cleanup
Nov 11, 2020
Merged

fix: normalize / test deprecation of 'Client.reserve_ids'#103
tseaver merged 5 commits intomasterfrom
101-reserve_ids-deprecation-cleanup

Conversation

@tseaver
Copy link
Contributor

@tseaver tseaver commented Oct 7, 2020

Closes #101
Closes #100

@tseaver tseaver requested review from chrisrossi and crwilcox October 7, 2020 20:55
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Oct 7, 2020
@tseaver tseaver mentioned this pull request Oct 7, 2020
4 tasks
@tseaver tseaver added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Oct 8, 2020
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Oct 8, 2020
@tseaver tseaver requested review from frankyn and removed request for frankyn October 8, 2020 18:00
@crwilcox crwilcox requested a review from a team November 3, 2020 23:59
@crwilcox
Copy link
Contributor

crwilcox commented Nov 4, 2020

@tseaver this hasn't been merged yet, curious if you think we should release a 1.9.1 with this so we can deprecate/simplify for 2.0? or if you think this is commonly used enough to justify deprecating in 2.0?

@tseaver
Copy link
Contributor Author

tseaver commented Nov 11, 2020

@crwilcox Given that 2.0.0 is out already, I guess we just merge this and leave the deprecation in place until 3.0.

@crwilcox
Copy link
Contributor

@tseaver that seems reasonable to me.

@tseaver tseaver merged commit 5851522 into master Nov 11, 2020
@tseaver tseaver deleted the 101-reserve_ids-deprecation-cleanup branch November 11, 2020 21:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Excess comma in reserve_ids breaks the function

3 participants