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.

Sign up for GitHub

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

Jump to bottom

Cleanup forward compatibility of random ops #47896

Merged
merged 1 commit into from Apr 9, 2021
Merged

Cleanup forward compatibility of random ops #47896

merged 1 commit into from Apr 9, 2021

Conversation

lgeiger
Copy link
Contributor

@lgeiger lgeiger commented Mar 18, 2021

The forward compatibility horizons are expired, so these checks can be safely removed.

@lgeiger
Cleanup forward compatibility of random ops
058dda4 
The forward compatibility horizon expired so these checks can be savely 
removed
@google-ml-butler google-ml-butler bot added the size:L CL Change Size: Large label Mar 18, 2021
@google-cla google-cla bot added the cla: yes label Mar 18, 2021
@gbaned gbaned self-assigned this Mar 19, 2021
@gbaned gbaned added this to Assigned Reviewer in PR Queue via automation Mar 19, 2021
@gbaned gbaned requested a review from wangpengmit April 7, 2021 18:14
@gbaned gbaned added the awaiting review Pull request awaiting review label Apr 7, 2021
PR Queue automation moved this from Assigned Reviewer to Approved by Reviewer Apr 8, 2021
wangpengmit
wangpengmit approved these changes Apr 8, 2021
View reviewed changes
Copy link
Member

@wangpengmit wangpengmit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the clean-up!

@google-ml-butler google-ml-butler bot added kokoro:force-run Tests on submitted change ready to pull PR ready for merge process labels Apr 8, 2021
@kokoro-team kokoro-team removed kokoro:force-run Tests on submitted change labels Apr 8, 2021
@gbaned gbaned removed the awaiting review Pull request awaiting review label Apr 8, 2021
@rthadur rthadur added ready to pull PR ready for merge process and removed ready to pull PR ready for merge process labels Apr 8, 2021
@copybara-service copybara-service bot merged commit 1197171 into tensorflow:master Apr 9, 2021
PR Queue automation moved this from Approved by Reviewer to Merged Apr 9, 2021
@lgeiger lgeiger deleted the cleanup-compat branch April 9, 2021 15:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wangpengmit wangpengmit wangpengmit approved these changes

@joker-eph joker-eph Awaiting requested review from joker-eph

Assignees

@gbaned gbaned

Labels
cla: yes ready to pull PR ready for merge process size:L CL Change Size: Large
Projects
PR Queue
  
Merged
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@lgeiger @wangpengmit @kokoro-team @rthadur @gbaned