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

[ci] Adding .travis.yml #26

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

[ci] Adding .travis.yml #26

wants to merge 1 commit into from

Conversation

venantius
Copy link
Collaborator

This commit adds a .travis.yml, which @tirkarthi tested on his fork successfully here:

@venantius
Copy link
Collaborator Author

venantius commented Jan 20, 2018

@ninjudd It seems I need admin permissions in order to set up CI. If you don't want to grant admin permissions (which I understand), I'll need you to add this project to Travis using their UI before tests can run automatically.

@bbatsov
Copy link

bbatsov commented Mar 8, 2018

It's nice to see someone trying to improve things here, but given how much better compliment is, and that clojure-complete hasn't seen almost no development in years, maybe it'd be best to just put a deprecation notice and point people in the direction of compliment.

@venantius
Copy link
Collaborator Author

@bbatsov I completely agree - the issue at the moment is that Lein still relies on clojure-complete and someone will need to take the work on of migrating it over.

@bbatsov
Copy link

bbatsov commented Mar 8, 2018

Technically speaking it's not lein, it's reply (which lein uses for lein repl). I've started work on this a long time ago, but I never had the time to finish it trptcolin/reply#153

@venantius
Copy link
Collaborator Author

venantius commented Mar 8, 2018 via email

@tirkarthi
Copy link
Contributor

Might be a little off-topic but there were also some issues with compliment in JDK 9 over classpath. I have filed an issue at alexander-yakushev/compliment#52 .

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.

3 participants