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

Make left clicking on test open test by default #111

Conversation

davidpendraykalibrate
Copy link
Contributor

User can configure this behaviour via leftClickAction to either:

  • runTest
  • goToTest
  • doNothing
    When user clicks a test, this is initially handled by leftClickTest
    command which checks configuration and then passes request to
    appropriate handler. This ensures that when user changes configuration,
    the next click is handled correctly.

Replaces #108

@stefanforsberg
Copy link
Collaborator

Looks good! Mind fixing the tslint errors? You can do a npm run tslint in the root folder to see the errors locally (there's also a tslint extensions for vscode).

User can configure this behaviour via leftClickAction to either:
- runTest
- goToTest
- doNothing
When user clicks a test, this is initially handled by leftClickTest
  command which checks configuration and then passes request to
  appropriate handler. This ensures that when user changes configuration,
  the next click is handled correctly.
@davidpendraykalibrate
Copy link
Contributor Author

Done and got rid of merge conflict

@stefanforsberg stefanforsberg merged commit 12b1840 into formulahendry:master Aug 9, 2018
@davidpendraykalibrate davidpendraykalibrate deleted the NewLeftClickGoToTest branch August 9, 2018 08:25
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.

2 participants