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

Integrate Source maps? #67

Open
cengebretson opened this issue Mar 14, 2013 · 4 comments
Open

Integrate Source maps? #67

cengebretson opened this issue Mar 14, 2013 · 4 comments

Comments

@cengebretson
Copy link
Member

Not 100% sure this is doable, but it looks like its worth checking out. Probably will need to upgrade to uglify2

https://github.com/mishoo/UglifyJS2#source-map-options

@scottcorgan
Copy link

+1 PLEASE

@aeischeid
Copy link
Member

It's decently hard challenge to do source maps on these concatenated files, but is on the radar for sure

@adammunoz
Copy link

I think using https://github.com/michaelficarra/commonjs-everywhere would simplify things. Of course, it means changing the current implementation of hem based on stitch.

@cengebretson
Copy link
Member Author

I definitely feel like Hem needs to have some internal things redone. Just off the top of my head..

  1. Some of its dependencies are no longer actively worked on, need to find replacements
  2. Have a more convention over configuration approach, at least in regards to how a spine app is setup. Possibly integrate the spine.app functionality too.
  3. Incorporate some new features, I think the source mapping and some livereload functionality would be great. Need polish off the phantomjs/karma system for making it easier to test spine apps.
  4. Write tests for hem itself :o)

Hoping by mid-may I can get some time to work on a refreshed version of hem. I'll probably end up creating a milestone and separate branch for it fairly soon if you want to keep track of it. I know there are other build tools out there in the javascript world, but I still think there is room left for something that is smaller/simple and integrates seamlessly with spine.

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

No branches or pull requests

4 participants