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

update rake tasks #137

Merged
merged 1 commit into from
Oct 28, 2014
Merged

update rake tasks #137

merged 1 commit into from
Oct 28, 2014

Conversation

tonini
Copy link
Contributor

@tonini tonini commented Oct 28, 2014

Hi,

I just updated the Rakefile with two new tasks.

rake info     # Installed Emacs information
rake test     # Run the test suite

rake default task is now set to test

nothing fancy or special, just handy to run the test suite quick.

@tonini
Copy link
Contributor Author

tonini commented Oct 28, 2014

@mattdeboard maybe it would be good when you run them localy, to see how they do.

@mattdeboard
Copy link
Contributor

Can you modify rake test to accept command line args you'd pass to ert-runner? I use the --win option a lot when there are unexpected failures.

@mattdeboard
Copy link
Contributor

In other words just accept argv[2:] or whatnot

@tonini
Copy link
Contributor Author

tonini commented Oct 28, 2014

Ok I see. Is it enough when I make it possible to run --win or --no-win. I think for the rest you should just use cask exec ert-runner directly.

@mattdeboard
Copy link
Contributor

Yeah I guess like a -i flag to the test to use --win should be fine.

@mattdeboard
Copy link
Contributor

Oh, actually, you know, you're right I think. It's fine without the option. Using cask exec ert-runner direct should suffice.

mattdeboard added a commit that referenced this pull request Oct 28, 2014
@mattdeboard mattdeboard merged commit 7693331 into master Oct 28, 2014
@mattdeboard mattdeboard deleted the refine-rake-task branch October 28, 2014 20:38
@tonini
Copy link
Contributor Author

tonini commented Oct 28, 2014

👍

J3RN pushed a commit to J3RN/emacs-elixir that referenced this pull request Apr 24, 2021
Also moved SymbolInformation and DocumentSymbol under
ElixirLS.LanguageServer.Protocol

Added a simple test for the Definition provider
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