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

Next step require #414

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Next step require #414

wants to merge 7 commits into from

Conversation

aleksanb
Copy link
Member

No description provided.

@aleksanb aleksanb force-pushed the next-step-require branch 2 times, most recently from 383aaee to 76bd21d Compare April 24, 2017 22:28
@sigvef
Copy link
Member

sigvef commented Apr 24, 2017

needs rebase

This keeps the namespace and import tree clean,
as single files with just one class definition like TextureInput and
TextureOutput were a bit overkill.
@cristeahub
Copy link
Member

Why is the closure compiler commented out for the time being?

I also did a small comment on e934b92 , and the question here is what are you guys doing and how can I track it? There's no issues or discussions on what has been done I believe, so it's hard for me to contribute right now.

If they are needed, they can be re-added manually to each demo project's
lib folder.
This slims down bootstrap.js and makes my task of requiring up
bootstrap.js easier.

They're not used in the demo 're', so we don't have to maintain any
potential backwards compatability.
@sigvef
Copy link
Member

sigvef commented Jul 30, 2017

needs rebase

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

3 participants