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.

Sign up for GitHub

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

Jump to bottom

Field files split #710

Merged
merged 2 commits into from
Nov 2, 2021
Merged

Field files split #710

merged 2 commits into from
Nov 2, 2021

Conversation

azmeuk
Copy link
Member

@azmeuk azmeuk commented Oct 28, 2021

I splitted fields/core.py and fields/simple.py in several, smaller files. I think this makes the whole thing more readable and contributor friendly.

I would like to merge this before releasing 3.0

@azmeuk azmeuk added the refactoring Does the same thing but in a different way label Oct 28, 2021
@azmeuk azmeuk requested a review from davidism October 28, 2021 15:12
@azmeuk azmeuk merged commit 563b781 into wtforms:master Nov 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@davidism davidism Awaiting requested review from davidism

Assignees
No one assigned
Labels
refactoring Does the same thing but in a different way
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@azmeuk