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.

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

Allow labels to include an index via %i #773

Merged
merged 1 commit into from Jan 6, 2012
Merged

Allow labels to include an index via %i #773

merged 1 commit into from Jan 6, 2012

Conversation

mhuggins
Copy link
Contributor

@mhuggins mhuggins commented Jan 5, 2012

This pull request solves issue #771 that I opened several days ago.

justinfrench added a commit that referenced this pull request Jan 6, 2012
Allow labels to include an index via %i
@justinfrench justinfrench merged commit 5641f43 into formtastic:master Jan 6, 2012
justinfrench added a commit that referenced this pull request Jan 6, 2012
This reverts commit 5641f43, reversing
changes made to 1df5245.

Seeing if the build passes like it did previously before this was merged in.
justinfrench added a commit that referenced this pull request Jan 14, 2012
This reverts commit 5641f43, reversing
changes made to 1df5245.
@justinfrench
Copy link
Member

@mhuggins I've reverted this pull request in master because master's been unstable for a few days and I'm not confident I can make the right changes to keep it stable and accurate based o \n your work so far. Would still love to see this patch, so maybe get your repo in line with master, run bundle exec rake spec (it'll test against multiple Rails version) and send another pull request when you're happy. Many thanks for your work so far!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants