Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

exclude cdash url from url check #1604

Merged
merged 1 commit into from
Aug 29, 2023

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

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

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