docs.scala-lang icon indicating copy to clipboard operation
docs.scala-lang copied to clipboard

Only check external links daily

Open ashawley opened this issue 6 years ago • 7 comments

Seth suggested moving the fix in https://github.com/scala/scala-lang/pull/942 that modified Travis to check external links only on a scheduled basis, but still check internal links with all git activity.

This repo doesn't use Travis. Drone doesn't appear to intrinsically support cron jobs. There would need to be a similar facility to port it here.

ashawley avatar Aug 04 '18 22:08 ashawley

@jvican is this an issue you've faced/solved in any of the Scala Center's other Drone builds?

SethTisue avatar Aug 06 '18 21:08 SethTisue

No, never. Adding a cron job in our machine shouldn't be an issue, but I'm not sure this is the way to go... now we're bothered by it and we at least fix them. With such a tool, I'm not sure if they would even get fixed...

jvican avatar Aug 06 '18 21:08 jvican

No, never. Adding a cron job in our machine shouldn't be an issue, but I'm not sure this is the way to go... now we're bothered by it and we at least fix them. With such a tool, I'm not sure if they would even get fixed...

this has to be weighed against the cost of routinely having spurious errors in PRs that are unrelated to the content of the PRs. that leads to CI being ignored by contributors and maintainers. this is the situation we are already in.

SethTisue avatar Aug 06 '18 21:08 SethTisue

is there a strong reason we're using Drone instead of Travis-CI in this repo at all? generally we prefer to 1) keep the scala-lang and docs.scala-lang repos aligned technically, 2) use tools that are already familiar to open-source contributors (and under scala/* we've been using Travis-CI almost everywhere for a long time)

SethTisue avatar Aug 06 '18 21:08 SethTisue

The reason to use Drone was that it was faster to build the website (before it took ages). However, maybe now Travis has sped up and it's on par with Drone. If somebody wants to give it a try, feel free to!

that leads to CI being ignored by contributors and maintainers. this is the situation we are already in.

I see your point. I don't know if people ignore the CI because of this. If they do, that's bad. We can surely consider the option you guys propose.

jvican avatar Aug 07 '18 07:08 jvican

FYI, an exception was added to the CI build for links in the www.oracle.com domain, which were failing regularly. (Issue #1168, fixed in PR #1169) It doesn't appear verifying external links is otherwise an issue at the moment.

rkoeninger avatar Oct 10 '18 18:10 rkoeninger

Agree there's nothing really pressing here. (I still think it would be nice to align with the scala/scala-lang repo, though.)

SethTisue avatar Oct 10 '18 18:10 SethTisue