-
Couldn't load subscription status.
- Fork 147
Added configuration "Compile" to command that sets mainClass. #239
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
Conversation
| Can you squash the commits into one. |
doc/project/building.md Outdated
| If you want the code which is used to run the main class to be written to a file, you can set `persistLauncher := true`. | ||
| Note that this will require your main class to be either unique or explicitly set (`mainClass := Some(<name>)`). | ||
| Note that this will require your main class to be either unique or explicitly set (`mainClass in Compile := Some(<name>)`). | ||
| If you explicitly set `mainClass`, note that it needs to be set on a per-configuration basis (i.e. the part `in Compile` is essential else the setting will be ignored). For further detail see [Stack overflow entry 'How to set mainClass in ScalaJS build.sbt?'](http://stackoverflow.com/questions/34965072/how-to-set-mainclass-in-scalajs-build-sbt) (specific to Scala.js) and [Stack Overflow entry 'How to set main class in build?'](http://stackoverflow.com/questions/6467423/how-to-set-main-class-in-build) (not specific to Scala.js). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few improvements to the text:
essential else , otherwise the setting
For further detail information see the Stack Overflow entry [...] and the Stack Overflow entry
| It would also be nice to introduce a blank line (i.e., 2 new lines) after the first |
| Hi - I'm a git noob. As you can see from the series of commits I did not have an md viewer last night. I also made the commits on my master. Not sure how to squash them now. Tried some basic commands but got a message saying my attempt was the noop case. |
| You'll need to use interactive rebase and then after squashing is done, push the changes with |
| A good read: https://sandofsky.com/blog/git-workflow.html |
Also added a brief explanation and links to two relevant Stack Overflow questions.
| Thanks for the tips and the article, which was good. I have now got everything in one commit on a new branch called clean_attempt2. Should I create a new pull request from that branch? If I do, I guess you will have your desired effect. But I will still be left with the problem, should I want to make any further contributions, that master on my fork is polluted. Note that
I have been having fun on the command line: |
| If |
| LGTM Thanks! |
Added configuration "Compile" to command that sets mainClass.
| Just a minute... I've been flailing but have a plan. |
| lol sorry guys - I assumed that my own fork's master was what you called "the branch of this PR.". So I ran I was surprised to see that it lopped off all my commits in My error was leaving out the colon.
|
| Well, I'm not sure what you did, but the commit I merged was fine. It was one single commit, with a proper commit message and the proper content. You can see that in the Commits tab of this page (up above). As far as I'm concerned, you did well. |
| Thanks. Yes, I saw the commit at the top of this page, but when I go to https://github.com/scala-js/scala-js-website/commits/master... well there was nothing, but it (my change) has now appeared. Perhaps it was on your local machine for a while there. I don't know. Hopefully this will be a little smoother next time! Thanks for the help. |
One thing that helps is to always create PRs out of separate branches. Never from master. |
Added configuration "Compile" to command that sets mainClass. Also added a brief explanation and links to two relevant Stack Overflow questions.