Change verb from 'inject' to 'bind'. #243

Merged
merged 1 commit into from Mar 15, 2015

Projects

None yet

2 participants

@JakeWharton
Owner
  • InjectView becomes FindView
  • InjectViews becomes FindViews
  • ButterKnife.inject() becomes ButterKnife.bind()
  • ButterKnife.reset() becomes ButterKnife.unbind()
  • Previously deprecated Nullable annotation is removed.
@JakeWharton Change verb from 'inject' to 'bind'.
* InjectView becomes FindView
* InjectViews becomes FindViews
* ButterKnife.inject() becomes ButterKnife.bind()
* ButterKnife.reset() becomes ButterKnife.unbind()
* Previously deprecated Nullable annotation is removed.
ec778ad
@artem-zinnatullin
Contributor

O_o finally 👍

grep -rl "inject" ./*"

gives this:

./README.md
./butterknife/src/main/java/butterknife/ButterKnife.java
./butterknife/src/main/java/butterknife/internal/Binding.java
./butterknife/src/main/java/butterknife/internal/ButterKnifeProcessor.java
./butterknife/src/main/java/butterknife/internal/ViewInjector.java
./butterknife/src/main/java/butterknife/Nullable.java
./butterknife/src/main/java/butterknife/package-info.java
./butterknife/src/test/java/butterknife/ButterKnifeTest.java
./butterknife/src/test/java/butterknife/internal/AllTheThingsTest.java
./butterknife/src/test/java/butterknife/internal/InjectViewsTest.java
./butterknife/src/test/java/butterknife/internal/InjectViewTest.java
./butterknife/src/test/java/butterknife/internal/OnCheckedChangedTest.java
./butterknife/src/test/java/butterknife/internal/OnClickTest.java
./butterknife/src/test/java/butterknife/internal/OnEditorActionTest.java
./butterknife/src/test/java/butterknife/internal/OnFocusChangeTest.java
./butterknife/src/test/java/butterknife/internal/OnItemClickTest.java
./butterknife/src/test/java/butterknife/internal/OnItemLongClickTest.java
./butterknife/src/test/java/butterknife/internal/OnItemSelectedTest.java
./butterknife/src/test/java/butterknife/internal/OnLongClickTest.java
./butterknife/src/test/java/butterknife/internal/OnPageChangeTest.java
./butterknife/src/test/java/butterknife/internal/OnTextChangedTest.java
./butterknife/src/test/java/butterknife/internal/OnTouchTest.java
./butterknife-sample/src/main/java/com/example/butterknife/SimpleActivity.java
./butterknife-sample/src/main/java/com/example/butterknife/SimpleAdapter.java
./pom.xml
./website/ide-eclipse.html
./website/ide-idea.html
./website/index.html

Or if this PR will be merged, I can make PR with fixes for these files

@JakeWharton
Owner

All of the ones worth fixing are already fixed. I didn't bother with the tests.

@JakeWharton JakeWharton merged commit 34d3082 into master Mar 15, 2015

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
@JakeWharton JakeWharton deleted the jw/bind branch Mar 15, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment