Update to 1.9 syntax for hashes #313

Closed
steveklabnik opened this Issue Nov 12, 2012 · 6 comments

Comments

Projects
None yet
2 participants
@steveklabnik
Contributor

steveklabnik commented Nov 12, 2012

Rails especially is now using 1.9 everywhere as of Rails 4, so we should convert over at some point. Especially since we introduce hashes after symbols.

@kytrinyx

This comment has been minimized.

Show comment Hide comment
@kytrinyx

kytrinyx Nov 12, 2012

Contributor

Should we also add a section that describes the old-school hash syntax and explain that you can have non-symbol keys?

Contributor

kytrinyx commented Nov 12, 2012

Should we also add a section that describes the old-school hash syntax and explain that you can have non-symbol keys?

@steveklabnik

This comment has been minimized.

Show comment Hide comment
@steveklabnik

steveklabnik Nov 12, 2012

Contributor

That's probably a good idea.

Contributor

steveklabnik commented Nov 12, 2012

That's probably a good idea.

@steveklabnik

This comment has been minimized.

Show comment Hide comment
@steveklabnik

steveklabnik Nov 12, 2012

Contributor

This isn't truly done yet, because we need to change all the existing code to the new style too.

Contributor

steveklabnik commented Nov 12, 2012

This isn't truly done yet, because we need to change all the existing code to the new style too.

@kytrinyx

This comment has been minimized.

Show comment Hide comment
@kytrinyx

kytrinyx Nov 12, 2012

Contributor

Got it :)

Contributor

kytrinyx commented Nov 12, 2012

Got it :)

@kytrinyx

This comment has been minimized.

Show comment Hide comment
@kytrinyx

kytrinyx Nov 14, 2012

Contributor

@steveklabnik What about active record validations? The rails guides use the old-school hash syntax

Contributor

kytrinyx commented Nov 14, 2012

@steveklabnik What about active record validations? The rails guides use the old-school hash syntax

@kytrinyx kytrinyx closed this Nov 25, 2012

mikesea pushed a commit to mikesea/curriculum that referenced this issue Mar 25, 2014

Merge pull request #317 from kytrinyx/modernize-hashes
Introduce 1.9 hash syntax (100 minutes). Fixes #313

mikesea pushed a commit to mikesea/curriculum that referenced this issue Mar 25, 2014

mikesea pushed a commit to mikesea/curriculum that referenced this issue Mar 25, 2014

mikesea pushed a commit to mikesea/curriculum that referenced this issue Mar 25, 2014

mikesea pushed a commit to mikesea/curriculum that referenced this issue Mar 25, 2014

mikesea pushed a commit to mikesea/curriculum that referenced this issue Mar 25, 2014

mikesea pushed a commit to mikesea/curriculum that referenced this issue Mar 25, 2014

mikesea pushed a commit to mikesea/curriculum that referenced this issue Mar 25, 2014

mikesea pushed a commit to mikesea/curriculum that referenced this issue Mar 25, 2014

mikesea pushed a commit to mikesea/curriculum that referenced this issue Mar 25, 2014

mikesea pushed a commit to mikesea/curriculum that referenced this issue Mar 25, 2014

mikesea pushed a commit to mikesea/curriculum that referenced this issue Mar 25, 2014

mikesea pushed a commit to mikesea/curriculum that referenced this issue Mar 25, 2014

mikesea pushed a commit to mikesea/curriculum that referenced this issue Mar 25, 2014

mikesea pushed a commit to mikesea/curriculum that referenced this issue Mar 25, 2014

mikesea pushed a commit to mikesea/curriculum that referenced this issue Mar 25, 2014

mikesea pushed a commit to mikesea/curriculum that referenced this issue Mar 25, 2014

mikesea pushed a commit to mikesea/curriculum that referenced this issue Mar 25, 2014

mikesea pushed a commit to mikesea/curriculum that referenced this issue Mar 25, 2014

mikesea pushed a commit to mikesea/curriculum that referenced this issue Mar 25, 2014

mikesea pushed a commit to mikesea/curriculum that referenced this issue Mar 25, 2014

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