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

Add deprecated annotation to untyped APIs #96

Merged
merged 7 commits into from
Jul 15, 2021

Conversation

negokaz
Copy link
Contributor

@negokaz negokaz commented Jul 15, 2021

No description provided.

@tksugimoto tksugimoto force-pushed the untyped-api-is-deprecated branch from 48786b2 to 025c51c Compare July 15, 2021 06:41
negokaz and others added 2 commits July 15, 2021 15:44
…lassic API)

Because it is used internally.
ClusterReplicationSetting is always used with ClusterReplication, so there is no problem without deprecated annotation.
@tksugimoto tksugimoto force-pushed the untyped-api-is-deprecated branch from 025c51c to 11bdd76 Compare July 15, 2021 07:08
Because it is a test of deprecated API
@tksugimoto tksugimoto force-pushed the untyped-api-is-deprecated branch from 11bdd76 to 4ceaaa8 Compare July 15, 2021 07:20
@tksugimoto tksugimoto self-requested a review July 15, 2021 07:21
tksugimoto
tksugimoto previously approved these changes Jul 15, 2021
- Easy to understand
- Unrelated warnings are not ignored
tksugimoto
tksugimoto previously approved these changes Jul 15, 2021
tksugimoto
tksugimoto previously approved these changes Jul 15, 2021
@tksugimoto tksugimoto changed the title 👷chore: Add @deprecated to untyped APIs Add deprecated annotation to untyped APIs Jul 15, 2021
@tksugimoto tksugimoto enabled auto-merge July 15, 2021 09:30
@tksugimoto tksugimoto mentioned this pull request Jul 15, 2021
1 task
@tksugimoto tksugimoto merged commit d89d153 into master Jul 15, 2021
@tksugimoto tksugimoto deleted the untyped-api-is-deprecated branch July 15, 2021 12:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants