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

Fix for the strip_stars regex #2

Closed
wants to merge 15 commits into from
Closed

Fix for the strip_stars regex #2

wants to merge 15 commits into from

Conversation

stvno
Copy link
Contributor

@stvno stvno commented Jul 5, 2011

The problem with the regex was that / a special character is in javascript regex. As such you have to use \057 I tested it on my code and jodoc-js now gives the same output as joDoc

@dfreedm
Copy link
Owner

dfreedm commented Jul 6, 2011

I think there's a bit too much in here that's custom for you, so I'll just merge in the regex fix. Thanks!

@dfreedm dfreedm closed this Jul 6, 2011
@stvno
Copy link
Contributor Author

stvno commented Jul 6, 2011

Hm only the regex fix was meant for the pull request, everything else is indeed completely custom. I'm still a GitHub n00b so probably did something wrong there.

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.

2 participants