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 Flute::Form->finalize #134

Merged
merged 1 commit into from Sep 4, 2015

Conversation

Projects
None yet
2 participants
@paultcochrane
Contributor

paultcochrane commented Aug 23, 2015

This method isn't used anywhere else in the code. In fact, a git grep finalize on the codebase showed that the word finalize only occured in the definition of this method and in its documentation. Since it doesn't seem to be needed, my recommendation is to remove it.

Remove Flute::Form->finalize
This method isn't used anywhere else in the code.  In fact, a `git grep
finalize` on the codebase showed that the word `finalize` only occured in
the definition of this method and in its documentation.  Since it doesn't
seem to be needed, my recommendation is to remove it.

racke added a commit that referenced this pull request Sep 4, 2015

@racke racke merged commit 950017c into racke:master Sep 4, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment