appsforartists/ambidex

Performance Tuning

Open

#13 aberto em 13 de mar. de 2015

Ver no GitHub
 (8 comments) (0 reactions) (0 assignees)JavaScript (582 stars) (31 forks)batch import
help wantedinvestigate

Description

Considering we are using webpack, maybe we should use it in order to remove this:

require("node-jsx").install(
  {
    "extension":  ".js",
    "harmony":    true
  }
);

at every init.js of projects that use ambidex.

Any thoughts?

Guia do colaborador

Pilha de tecnologia
javascriptnodejsreact
Domain
build systemperformance
Tipo Issue
performance
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 hours
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
stale
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
mostly clear
Prerequisites
webpack configurationnode jsxReact isomorphic app
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
35
Direção de pesquisa
This issue proposes removing the `require('node jsx')` call by leveraging webpack. To implement, first identify all `init.js` files that use node jsx. Then configure webpack to handle `.js` files with JSX syntax, typically via babel loader. The issue has no assignee and is open, so a solution can be proposed. Refer to the existing discussion in issue #13 for any prior considerations.