Skip to content
This repository was archived by the owner on Nov 9, 2017. It is now read-only.

Conversation

dasch
Copy link
Contributor

@dasch dasch commented Dec 9, 2012

Makes the regex more readable.

@dhh
Copy link
Member

dhh commented Feb 18, 2013

Please rebase.

Makes the regex more readable.
@dasch
Copy link
Contributor Author

dasch commented Feb 18, 2013

@dhh done.

What do you feel about #19?

dhh pushed a commit that referenced this pull request Feb 18, 2013
Format the regex with code delimiters
@dhh dhh merged commit 1bf04bb into rails:master Feb 18, 2013
@dhh
Copy link
Member

dhh commented Feb 18, 2013

I like the idea. Have you actually used this for a second dependency tracker, like for mustache or something else?

On Feb 18, 2013, at 4:26 PM, Daniel Schierbeck notifications@github.com wrote:

@dhh done.

What do you feel about #19?


Reply to this email directly or view it on GitHub.

@dasch dasch deleted the patch-1 branch February 18, 2013 17:08
@dasch
Copy link
Contributor Author

dasch commented Feb 18, 2013

@dhh I developed it for Curly, which is similar to Mustache in design. I haven't had it running for a while, I can try hooking it up again tomorrow. There's also an open PR on the Rails repo, but it hasn't received much attention.

@dhh
Copy link
Member

dhh commented Feb 18, 2013

I'd definitely be happy to support this extension. Makes good sense to me.

On Feb 18, 2013, at 6:37 PM, Daniel Schierbeck notifications@github.com wrote:

@dhh I developed it for Curly, which is similar to Mustache in design. I haven't had it running for a while, I can try hooking it up again tomorrow. There's also an open PR on the Rails repo, but it hasn't received much attention.


Reply to this email directly or view it on GitHub.

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

Successfully merging this pull request may close these issues.

2 participants