Skip to content
This repository has been archived by the owner on Aug 11, 2023. It is now read-only.

add linkcheck #482

Merged
merged 1 commit into from
Oct 28, 2020
Merged

add linkcheck #482

merged 1 commit into from
Oct 28, 2020

Conversation

nstarman
Copy link
Member

@nstarman nstarman commented Oct 4, 2020

In .tox there's a link check option. I've enabled this in the allow_failures section of .travis.yml.
Looking in the closed issues and PRs, I don't see anything mentioning the link check, so if this wasn't enable for a reason, I'll close this PR.

edit: I see some discussion of this in #449

Signed-off-by: Nathaniel Starkman nstarkman@protonmail.com

@nstarman nstarman marked this pull request as draft October 5, 2020 17:00
@pllim
Copy link
Member

pllim commented Oct 5, 2020

There is a cron example at https://github.com/astropy/astropy/blob/4bc50147063950cb43422243dcd8ac647f3abc12/.travis.yml#L71-L72 (search for "cron" on that file).

Also look at https://github.com/astropy/astropy/blob/4bc50147063950cb43422243dcd8ac647f3abc12/docs/conf.py#L255 .

Maybe also https://docs.travis-ci.com/user/cron-jobs/ .

Thanks! 😄

@nstarman nstarman marked this pull request as ready for review October 5, 2020 19:19
@nstarman nstarman mentioned this pull request Oct 5, 2020
@nstarman nstarman requested review from pllim and bsipocz October 8, 2020 19:08
Copy link
Member

@pllim pllim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks! Might want to squash the commits into one commit though.

I'll leave this open for a bit in case @astrofrog , @bsipocz , or @Cadair want to review.

{{ cookiecutter.package_name }}/.travis.yml Outdated Show resolved Hide resolved
add linkcheck to cron
changelog

Co-authored-by: P. L. Lim <2090236+pllim@users.noreply.github.com>

Signed-off-by: Nathaniel Starkman <nstarkman@protonmail.com>
@nstarman
Copy link
Member Author

nstarman commented Oct 25, 2020

@pllim @bsipocz , squashed. Thanks for working with me on this.

Copy link
Member

@pllim pllim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks!

@pllim pllim merged commit ff5fbde into astropy:cookiecutter Oct 28, 2020
@pllim pllim added this to the 3.1 milestone Oct 28, 2020
@nstarman nstarman deleted the linkcheck branch October 29, 2020 05:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants