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

Add some more notes about using the IDE on Windows #137

Closed
wants to merge 1 commit into from

Conversation

mhammond
Copy link
Member

@mhammond mhammond commented Nov 3, 2016

I made some notes when installing and trying to use IntelliJ which I've captured here.


This change is Reviewable

@mozilla-autolander-deprecated

Autolander could not find a bug number in your pull request title. All pull requests should be in the format of: Bug [number] - [description].

@codecov-io
Copy link

codecov-io commented Nov 3, 2016

Current coverage is 58.29% (diff: 100%)

Merging #137 into master will not change coverage

@@             master       #137   diff @@
==========================================
  Files            23         23          
  Lines          1976       1976          
  Methods        1837       1837          
  Messages          0          0          
  Branches         95         95          
==========================================
  Hits           1152       1152          
  Misses          824        824          
  Partials          0          0          

Powered by Codecov. Last update fbc5e28...9373306

@Dexterp37
Copy link
Contributor

Drive-by: @mhammond, maybe you should integrate this at the end of the README? I've added a Windows issues a while back that reports some of the things you've added.

@mhammond
Copy link
Member Author

mhammond commented Nov 3, 2016

maybe you should integrate this at the end of the README?

I prefer having the windows notes on the IDE close to the other notes about the IDE - the other Windows notes also seem relevant to the immediate section above it. I'm happy to move it if that's really wanted though (or have it re-jigged in-place as part of merging it :)

@Dexterp37
Copy link
Contributor

Yeah, I mean, I don't particularly care I'm just saying that the two sections should probably be merged (as they suggest the same solutions!)

@vitillo vitillo closed this Jan 25, 2017
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.

None yet

5 participants