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

models: Remove unnecessary transforms #2797

Merged
merged 1 commit into from Sep 18, 2020
Merged

Conversation

Turbo87
Copy link
Member

@Turbo87 Turbo87 commented Sep 17, 2020

These are only needed when API fields should be transformed from one type to another (e.g. from ISO8601 string to Date), but for the regular cases they don't need to be used.

r? @locks

@locks
Copy link
Contributor

locks commented Sep 18, 2020

@bors r+

@bors
Copy link
Contributor

bors commented Sep 18, 2020

📌 Commit f08acb9 has been approved by locks

@bors
Copy link
Contributor

bors commented Sep 18, 2020

⌛ Testing commit f08acb9 with merge 5872e3a...

@bors
Copy link
Contributor

bors commented Sep 18, 2020

☀️ Test successful - checks-travis
Approved by: locks
Pushing 5872e3a to master...

@bors bors merged commit 5872e3a into rust-lang:master Sep 18, 2020
@Turbo87 Turbo87 deleted the transforms branch September 18, 2020 12:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants