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

Set default FileWatch command to force a rebuild #1023

Merged

Conversation

@tedkornish
Copy link
Contributor

@tedkornish tedkornish commented Sep 21, 2015

Just something which could be kind of nice.

@snoyberg
Copy link
Contributor

@snoyberg snoyberg commented Sep 21, 2015

I'm OK with this, but I think it should still let the user know about the help command for discoverability. (Yes, it mentions help at the beginning, but that's easy to miss/forget.) Maybe just add it to the "waiting for next build" line?

@tedkornish
Copy link
Contributor Author

@tedkornish tedkornish commented Sep 21, 2015

^ disregard: that's a mistake and not what you're looking for (fc2d25b). I'll fix the failing build.

I think it should still let the user know about the help command for discoverability
Maybe just add it to the "waiting for next build" line?

Could you clarify what you mean by "it"? As far as I'm aware, every instance of telling the user about the help command is still there.

@snoyberg
Copy link
Contributor

@snoyberg snoyberg commented Sep 21, 2015

I mean that pressing enter by itself today gives the user some guidance about what's going on. I can picture someone not understanding how to exit and just hitting enter, for example.

@snoyberg
Copy link
Contributor

@snoyberg snoyberg commented Sep 21, 2015

And you can ignore the AppVeyor build failures, I'm just getting that set up now.

@snoyberg snoyberg merged commit fc2d25b into commercialhaskell:master Sep 27, 2015
0 of 2 checks passed
0 of 2 checks passed
continuous-integration/appveyor AppVeyor build failed
Details
continuous-integration/travis-ci/pr The Travis CI build failed
Details
snoyberg added a commit that referenced this pull request Sep 27, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants