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

Remove debugger in dust-full-1.2.1.js #249

Merged
merged 1 commit into from
Mar 19, 2013
Merged

Remove debugger in dust-full-1.2.1.js #249

merged 1 commit into from
Mar 19, 2013

Conversation

sethkinast
Copy link
Contributor

Trivial commit; you left a debugger statement in after your last merge.

Feel free to just toss this pull request and make the change the next time you commit.

vybs added a commit that referenced this pull request Mar 19, 2013
Remove debugger in dust-full-1.2.1.js
@vybs vybs merged commit 95bea19 into linkedin:master Mar 19, 2013
@vybs
Copy link
Contributor

vybs commented Mar 19, 2013

thanks @sethkinast .

@jairodemorais boo, LOL

@jairodemorais
Copy link
Contributor

The format function is never used anyway :p

@vybs
Copy link
Contributor

vybs commented Mar 19, 2013

Now that you say so, we will use it soon:)

@jairodemorais
Copy link
Contributor

@vybs we should not merge changes in dust-full or dust-core. Remember that those files are generated.

@vybs
Copy link
Contributor

vybs commented Mar 20, 2013

@jairodemorais but why upload them in the PR in the first place, ?

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.

3 participants