Clojure

travis on core clojure

Details

  • Type: Enhancement Enhancement
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Declined
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: None
  • Labels:

Description

Today not exist travis (auto ci, tests running) active on https://github.com/clojure/clojure

Activity

Hide
Andy Fingerhut added a comment -

Someone else can comment in a more official capacity, but the Clojure team has been using Jenkins to perform automated running of tests for at least 5 years, with results reported here: https://build.clojure.org

I suspect they are content with that, and would not consider it an advantage to switch to a different mechanism.

Show
Andy Fingerhut added a comment - Someone else can comment in a more official capacity, but the Clojure team has been using Jenkins to perform automated running of tests for at least 5 years, with results reported here: https://build.clojure.org I suspect they are content with that, and would not consider it an advantage to switch to a different mechanism.
Hide
Alex Miller added a comment -

There are a variety of reasons why moving to Travis or other automated CI hosts would be challenging. Significant effort has been spent making the Jenkins environment safer and more capable over the last couple years and we are relatively happy with what it's doing.

Show
Alex Miller added a comment - There are a variety of reasons why moving to Travis or other automated CI hosts would be challenging. Significant effort has been spent making the Jenkins environment safer and more capable over the last couple years and we are relatively happy with what it's doing.
Hide
Alex Miller added a comment -

There is some value in having automated patch testing, but since we don't accept PRs right now on github, that doesn't really do much for us.

Show
Alex Miller added a comment - There is some value in having automated patch testing, but since we don't accept PRs right now on github, that doesn't really do much for us.
Hide
Thiago Avelino added a comment -

Thanks for the explanation.

Show
Thiago Avelino added a comment - Thanks for the explanation.

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: