Skip to content

Conversation

Gozala
Copy link

@Gozala Gozala commented Jan 5, 2013

I have being using travis.ci with bunch of other projects and it's being great to be able too see status of test run in the pull requests regardless of weather you submit a change or do a review. It would be really useful to have this for clojurescript too.

Tests do work on my fork right now:
https://travis-ci.org/Gozala/clojurescript

In addition to pulling this change github hook needs to be enabled (before the pull) to trigger tests runs on pull requests and pushes. Details on this can be found here:

http://about.travis-ci.org/docs/user/how-to-setup-and-trigger-the-hook-manually/

@ldenman
Copy link

ldenman commented Jan 17, 2013

Cool idea, I like it! +1

@swannodette
Copy link
Member

As with all official Clojure projects - ClojureScript does not accept pull requests. Please submit tickets and patches to JIRA. It's also nice to post to the clojure-dev list to make sure there's actually interest in something like this first. Thanks!

@Gozala
Copy link
Author

Gozala commented Jan 18, 2013

If clojurescript doesn't take pull requests this is pointless, since whole purpose of it to displays weather submitted pull request passes or fails them.

P.S.: Patch for any pull request is just addition of .patch to the url see:
https://github.com/clojure/clojurescript/pull/21.patch

@rwaldron
Copy link

@Gozala is absolutely right.

Also, JIRA has git support and git is designed for distributed source control management—it's trivial to accept changes via pull request, merge them to your local clone and push back to the JIRA master and github.

@SlexAxton
Copy link

You can also integrate post-commit hooks in here to hit tickets in JIRA. See: https://confluence.atlassian.com/display/JIRASTUDIO/Linking+GitHub+Activities+to+JIRA+Issues

I'm sure you could modify a pull-request hook to create a ticket in JIRA and then link the .patch of the pull request. Maybe someone could do that work the project? I think you'd get more involvement, which is always good! Travis CI is a really great thing to have to help maintain quality in the patches that other devs are sending in.

@swannodette
Copy link
Member

Any process to improve workflow would be great. But I will say patches involve running quite a few sanity checks including manually checking that the both the Rhino REPL & the Browser REPL are not broken and that there are no serious performance regressions. It's perhaps possible to automate these as well, but no one has stepped forward with the initial work. When those things are covered then I think something like Travis CI would be considerably more compelling / useful.

Any takers? :)

@Gozala
Copy link
Author

Gozala commented Jan 18, 2013

I was planning on getting to that too, but had to start somewhere. This felt like a good entry point.

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.

5 participants