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

Error building JSX "1. Hello World" example #232

Closed
fanqi opened this issue May 2, 2017 · 2 comments
Closed

Error building JSX "1. Hello World" example #232

fanqi opened this issue May 2, 2017 · 2 comments

Comments

@fanqi
Copy link

fanqi commented May 2, 2017

The downloaded JSX boilerplate throws an error while building.

  1. Download boilerplate-jsx
  2. Unzip, then go to boilerpate-jsx/1. Hello World
  3. Execute npm i, then npm run build. The result is:
➜  1. Hello World npm run build

> [email protected] build /Users/fanqi/Projects/WebstormProjects/metaljs-boilerplate-jsx/1. Hello World
> gulp build

[15:12:19] Using gulpfile ~/Projects/WebstormProjects/metaljs-boilerplate-jsx/1. Hello World/gulpfile.js
[15:12:19] Starting 'build'...
[15:12:19] Starting 'clean'...
[15:12:19] Finished 'clean' after 7.57 ms
[15:12:19] Starting 'css'...
[15:12:19] Starting 'soy'...
[15:12:19] Finished 'soy' after 25 ms
[15:12:19] Starting 'build:js'...
[15:12:19] Starting 'build:globals:js'...
Error in plugin 'metal'
Message:
    Cannot call a class as a function (While processing preset: "/Users/fanqi/Projects/WebstormProjects/metaljs-boilerplate-jsx/1. Hello World/node_modules/metal/lib/metal.js")
[15:12:19] Finished 'build:globals:js' after 362 ms
[15:12:19] Finished 'build:js' after 362 ms
[15:12:19] Finished 'css' after 500 ms
[15:12:19] Starting 'uglify'...
[15:12:19] Finished 'uglify' after 3.14 ms
[15:12:19] Finished 'build' after 514 ms

➜  1. Hello World node -v && npm -v
v6.10.2
3.10.10
@bryceosterhaus
Copy link
Member

Hey @fanqi, thanks for catching that. Looks like our jsx boilerplate was pretty out of date. I updated it with a newer version of metal-jsx. I sent a pull-request to fix the download link here bryceosterhaus/metaljs.com#40

@bryceosterhaus
Copy link
Member

Fix was merged and new zip is available at http://metaljs.com/docs/

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

2 participants