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

Replace #178

Merged
merged 6 commits into from Mar 21, 2017

Conversation

Projects
None yet
3 participants
@arrdem
Collaborator

arrdem commented Mar 21, 2017

Continues the work from #158, fixing the current merge conflicts.

Includes more recent additions to the driver ns's docstrings, and dependency updates.

Tests are clean.

jpb and others added some commits Apr 7, 2016

Automatically apply suggestions to source files
Introduces `--replace` and `--interactive` cli arguments to automatically
replace suggestions in source files.

Initial port from jpb/kibit-replace b49410c

Resolves #155
Don't have check/check-expr traverse the form
replace/check-expr already does a full depth traversal of
the form - previously every form was traversed n times
(n being the depth of the form).
Merge master into pull/158/head (@jpb's replace changes)
Conflicts:
  project.clj
  src/kibit/driver.clj
@arrdem

This comment has been minimized.

Show comment
Hide comment
@arrdem

arrdem Mar 21, 2017

Collaborator

Ping @danielcompton for a merge if this is green in CI.

Collaborator

arrdem commented Mar 21, 2017

Ping @danielcompton for a merge if this is green in CI.

@danielcompton danielcompton merged commit 66b20f1 into master Mar 21, 2017

2 checks passed

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