Skip to content

Remove redundant Codable implementation from Tag. #169

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

Merged
merged 1 commit into from
Jan 4, 2024

Conversation

grynspan
Copy link
Contributor

@grynspan grynspan commented Jan 4, 2024

The default synthesized implementation of Codable is equivalent to what we've manually implemented for Tag, so we can delete the manual implementation.

Checklist:

  • Code and documentation should follow the style of the Style Guide.
  • If public symbols are renamed or modified, DocC references should be updated.

The default synthesized implementation of `Codable` is equivalent to what we've manually implemented for `Tag`, so we can delete the manual implementation.
@grynspan
Copy link
Contributor Author

grynspan commented Jan 4, 2024

@swift-ci please test

@grynspan grynspan merged commit 03f7972 into main Jan 4, 2024
@grynspan grynspan deleted the jgrynspan/redundant-tag-codable-implementation branch January 4, 2024 20:51
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