Skip to content

exclude cdash url from url check#1604

Merged
shahzebsiddiqui merged 1 commit intodevelfrom
exclude_cdash_url_from_urlchecker
Aug 29, 2023
Merged

exclude cdash url from url check#1604
shahzebsiddiqui merged 1 commit intodevelfrom
exclude_cdash_url_from_urlchecker

Conversation

@shahzebsiddiqui
Copy link
Member

this PR will exclude the cdash url https://my.cdash.org/ from the url checker action since some of the paths like https://my.cdash.org//viewTest.php?buildid=2278337 will get broken over time.

@shahzebsiddiqui
Copy link
Member Author

@jscook2345 urlchecker is now passing https://github.com/buildtesters/buildtest/actions/runs/6013792381/job/16312012808?pr=1604 thanks for bringing this up

@shahzebsiddiqui shahzebsiddiqui merged commit 8f140bc into devel Aug 29, 2023
@shahzebsiddiqui shahzebsiddiqui deleted the exclude_cdash_url_from_urlchecker branch August 29, 2023 14:41
@jscook2345
Copy link

jscook2345 commented Aug 29, 2023

Looks like there was a warning in the readthedocs stuff, which should probably be fixed:

/home/docs/checkouts/readthedocs.org/user_builds/buildtest/checkouts/1604/docs/api/buildtest/defaults/index.rst:56: WARNING: duplicate object description of buildtest.defaults.VAR_DIR, other instance in api/buildtest/defaults/index, use :noindex: for one of them

@shahzebsiddiqui
Copy link
Member Author

Looks like there was a warning in the readthedocs stuff, which should probably be fixed:

/home/docs/checkouts/readthedocs.org/user_builds/buildtest/checkouts/1604/docs/api/buildtest/defaults/index.rst:56: WARNING: duplicate object description of buildtest.defaults.VAR_DIR, other instance in api/buildtest/defaults/index, use :noindex: for one of them

Yeah i am trying to figure this out. Ideally this should not be documented because these are just variables. You can see https://buildtest.readthedocs.io/en/devel/api/buildtest/defaults/index.html page.

The main issue is we have multiple declarations in same file. I am trying to find way to ignore the file tried using autoapi_ignore or autoapi_options to workaround this issue https://sphinx-autoapi.readthedocs.io/en/latest/reference/config.html

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants