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

Allow non-js extensions #6

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

thomasboyt
Copy link

This PR lets you set the extension for the path the jsx! filter looks for (personally, I use .jsx):

requirejs.config({
  // ...
  jsx: {
    extension: 'jsx'
  }
});

@pirelenito
Copy link

+1

@iamnoah
Copy link

iamnoah commented Mar 29, 2014

+1, was about to submit the same thing.

@seiffert
Copy link
Owner

Thank you for your contribution! I'm no longer using this plugin actively, but I can merge this PR if you want!
If you have further ideas for improvement, you could also fork this repository and I'll put a deprecation notice into README with a link to your repository.

@pirelenito
Copy link

Hello everyone.

I've been using jsx-requirejs-plugin in my RequireJS setups. There is jsx file extension support via a configuration parameter (much like this pull request).

It is being actively developed, so it might be worth taking a look.

@mrienstra
Copy link

@seiffert, might as well update the README.md to mention that you are looking for a successor, and also direct folks towards philix/jsx-requirejs-plugin.

@murashki
Copy link

murashki commented Jun 9, 2015

+1

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.

6 participants