Skip to content

Conversation

tseaver
Copy link
Contributor

@tseaver tseaver commented Nov 19, 2020

Closes #100

@tseaver tseaver requested a review from a team as a code owner November 19, 2020 20:17
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Nov 19, 2020
@tseaver tseaver changed the title fix: undprecate entity factory helpers fix: undeprecate entity factory helpers Nov 30, 2020
@parthea
Copy link
Collaborator

parthea commented Jun 25, 2021

Hi @tseaver ,

Can you please confirm if this PR is still relevant?

@tseaver
Copy link
Contributor Author

tseaver commented Jun 25, 2021

@parthea

Can you please confirm if this PR is still relevant?

Yes, I've just not been able to get anyone's attention to review it.

@tseaver
Copy link
Contributor Author

tseaver commented Jun 25, 2021

I made the argument over in issue #100 that these deprecations were done in error: they were part of a change which deprecated one thing for real, but was over-zealous in the execution.

@tseaver tseaver merged commit 1fbee03 into master Jun 30, 2021
@tseaver tseaver deleted the 100-undeprecate-factory-helpers branch June 30, 2021 17:33
This was referenced May 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Undeprecate IAM factory helpers
2 participants