Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: undeprecate entity factory helpers #101

Merged
merged 2 commits into from Jun 30, 2021

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
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