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

Experiment with kotlinpoet without builders #487

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Experiment with kotlinpoet without builders #487

wants to merge 1 commit into from

Commits on Sep 30, 2018

  1. Experiment with kotlin-poet without builders 

    I thought the builder pattern was mostly useless in kotlin,
which has named parameters, default values and works well
with an immutable functional style like the one used by kotlin poet.

This is an experiment with the kotlinpoet codebase to test this hypothesis.

Of course I don't advocate removing the builders apis,
that would be bad for people calling kotlinpoet from java or with an existing code base.
I'm only adding stuff, and delegating to those builders,
even adding a configuration lambda for all the corner cases where a builder can be useful

I modified the examples in the README to see what the changes would look like
    jmfayard committed Sep 30, 2018
    Configuration menu
    Copy the full SHA
    27ca499 View commit details
    Browse the repository at this point in the history