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

Remove Obsolete items #1422

Merged
merged 4 commits into from
Jul 17, 2016
Merged

Remove Obsolete items #1422

merged 4 commits into from
Jul 17, 2016

Conversation

ryangribble
Copy link
Contributor

Our current approach to 🔥'ing deprecated items is to remove them a couple of releases after they've been marked [Obsolete]

The following [Obsolete] items which were marked in v0.19 or earlier are removed by this PR:

AuthorizationClient

  • RevokeAllApplicationAuthentications()

RepositoriesClient

  • CommitStatus
  • RepoCollaborators
  • Commits
  • RepositoryComments

GitHubClient

  • Release
  • Notification
  • GitDatabase

SshKey

  • SshKeysClient
  • SshKey
  • SshKeyInfo
  • SshKeyUpdate

Also adjusted/removed affected test classes

- RevokeAllApplicationAuthentications()
- CommitStatus
- RepoCollaborators
- Commits
- RepositoryComments
- Release
- Notification
- GitDatabase
- SshKeysClient
- SshKey
- SshKeyInfo
- SshKeyUpdate
@ryangribble ryangribble changed the title Out with the old Remove Obsolete items Jul 16, 2016
@shiftkey
Copy link
Member

@shiftkey shiftkey merged commit 75bd374 into octokit:master Jul 17, 2016
@ryangribble ryangribble deleted the out-with-the-old branch July 20, 2016 12:14
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