Skip to content

Comment out console.log#2

Closed
satchit8 wants to merge 1 commit intoquil:masterfrom
satchit8:patch-1
Closed

Comment out console.log#2
satchit8 wants to merge 1 commit intoquil:masterfrom
satchit8:patch-1

Conversation

@satchit8
Copy link
Copy Markdown

@satchit8 satchit8 commented Dec 6, 2016

Perhaps this will lead to less clutter in our consoles.

@nbeloglazov
Copy link
Copy Markdown
Member

processing.js is copied without modifications from original processing.js repo. So the next release will overwrite your changes anyway. You should submit pull request here instead: https://github.com/processing-js/processing-js

@satchit8
Copy link
Copy Markdown
Author

satchit8 commented Dec 8, 2016

Thanks for the consideration. I'll give it some thought.

@satchit8 satchit8 closed this Dec 8, 2016
@craigmalone
Copy link
Copy Markdown

I submitted an issue for this to https://github.com/processing-js/processing-js (processing-js/processing-js#275 (comment)).

It has now been fixed in processing.js 1.6.4, so next update to this library should pick it up.

@nbeloglazov
Copy link
Copy Markdown
Member

Thanks Craig. I'll do a new release soon. There also was a new release of processing itself so good time to cut a release.

@satchit8 satchit8 deleted the patch-1 branch February 6, 2017 07:36
@nbeloglazov
Copy link
Copy Markdown
Member

Released quil 2.6.0 and the issue should be fixed. Thanks @satchit8 and @craigmalone for bringing the issue up and following through with processing.js!

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