Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

remove unused require from AS/core_ext/class/delegating_att.rb #5331

Merged
merged 1 commit into from Mar 26, 2012

Conversation

Projects
None yet
3 participants
Contributor

vishnuatrai commented Mar 7, 2012

remove unused require. nolonger useful. with AS/core_ext/class/delegating_att.rb, now we are not using blank? and extract_options! in Class extension

Contributor

radar commented Mar 9, 2012

Please use more descriptive issue titles + commit messages please. In this case, telling us where you're removing the require from in the title would be great.

Contributor

vishnuatrai commented Mar 10, 2012

@radar 👍, done

@tenderlove tenderlove added a commit that referenced this pull request Mar 26, 2012

@tenderlove tenderlove Merge pull request #5331 from castlerock/remove_unused_require
remove unused require from AS/core_ext/class/delegating_att.rb
281b31b

@tenderlove tenderlove merged commit 281b31b into rails:master Mar 26, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment