-
Notifications
You must be signed in to change notification settings - Fork 617
Closed
Labels
good first issuehelp wantedNeeds help as a contributionNeeds help as a contributiontest-casesRelated to Addons testsRelated to Addons tests
Description
We enforce some rules in source_code_test.py about what API we can't use in addons. (no private api, no tf.compat.v1, and no experimental API in user-facing code).
We didn't enforce it in some files because they didn't respect the rules before the check was put into place. Replacing the problematic code takes some time, so this is why there are blacklists now. To track the progress. Replacing some APIs by public ones can be very easy or very hard depending on what the code is about. We'll gladly take pull request removing some elements from the blacklists. When we remove all elements, we can remove the blacklists entirely.
It's possible to copy the code needed into addons, or to find another similar API as long as the effect is the same.
Metadata
Metadata
Assignees
Labels
good first issuehelp wantedNeeds help as a contributionNeeds help as a contributiontest-casesRelated to Addons testsRelated to Addons tests