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

Initial WebStorm project #3041

Merged
merged 3 commits into from
Oct 14, 2015
Merged

Initial WebStorm project #3041

merged 3 commits into from
Oct 14, 2015

Conversation

mramato
Copy link
Contributor

@mramato mramato commented Sep 21, 2015

This adds a basic WebStorm project for devs that want to use WebStorm. It's not as robust as the Eclipse set up yet, but I'll remedy that in a future PR. For now:

  1. Configure code formatters to mimic Eclipse formatting. It's not 100% but it's close
  2. Add a File Watch that triggers ant build whenever a glsl file is modified.
  3. Mark Build directory as ignored, the root directory as resource root, and the Specs directory as test root.

ant is running the windows command right now, as there doesn't seem to be a good way to configure it cross platform, from what I can tell.

As an aside, I'm going create a roadmap to start to move us to gulp for any low hanging fruit, with the overall goal of moving away from ant completely.

This adds a basic WebStorm project for devs that want to use WebStorm.
It's not as robust as the Eclipse set up yet, but I'll remedy that in a
future PR.  For now:

1. Configure code formatters to mimic Eclipse formatting.  It's not 100%
but it's close

2. Add a File Watch that triggers `ant build` whenever a glsl file is
modified.

3. Mark `Build` directory as ignored, the root directory as resource root,
and the `Specs` directory as test root.
@mramato
Copy link
Contributor Author

mramato commented Sep 28, 2015

Can someone please take this for a spin?

@emackey
Copy link
Contributor

emackey commented Sep 28, 2015

I think @jkrrv did some testing with this when he was here. James are you still interested in testing a webstorm pull request?

@jkrrv
Copy link

jkrrv commented Sep 28, 2015

@emackey: I'd be happy to. It'll take me a couple days to get to it, though.

@hpinkos
Copy link
Contributor

hpinkos commented Oct 14, 2015

This looks good to me!
@mramato is this all set to be merged in? Should we add documentation to the wiki somewhere?

@mramato
Copy link
Contributor Author

mramato commented Oct 14, 2015

There's a separate issue to update the wiki and I plan on opening some additional pull requests this week to further advance webstorm integration (and our build process), so I'll probably do one big update at the end of the week.

@jkrrv
Copy link

jkrrv commented Oct 14, 2015

.gitignore should also have .idea/tasks.xml

Otherwise, looks good.

@mramato
Copy link
Contributor Author

mramato commented Oct 14, 2015

Thanks, I was unaware of tasks.xml but after double checking, I agree it should be ignored.

This is ready.

hpinkos pushed a commit that referenced this pull request Oct 14, 2015
@hpinkos hpinkos merged commit 0590911 into master Oct 14, 2015
@hpinkos hpinkos deleted the webstorm branch October 14, 2015 21:36
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.

4 participants