Remove linkcheck

[michaelcdevin]

Description

Resolves #286.

Type of PR

• Bug fix
• New feature
• Documentation
• Other: CI/CD

Checklist for PR

• Authors read the contribution guidelines
• The pull request is from an issue branch (not main) on your fork, to the main branch in WecOptTool.
• The authors have given the admins edit access
• All changes adhere to the style guide including PEP8, Docstrings, and Type Hints.
• Modified the documentation if applicable
• All tests pass
• [Reference or close any relevant issues](https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue

[coveralls]

Pull Request Test Coverage Report for Build 6649764870

• 0 of 0 changed or added relevant lines in 0 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+0.07%) to 92.524%

---

Totals
Change from base Build 6536704021:	0.07%
Covered Lines:	2500
Relevant Lines:	2702

---

💛 - Coveralls

[ryancoe]

@michaelcdevin - running python build_docs.py still appears to be checking links and just not throwing fatal errors, is that right?

WecOptTool/docs/build_docs.py

Lines 15 to 22 in bdf192c

	def linkcheck():
	app = Sphinx(source_dir,
	conf_dir,
	linkcheck_dir,
	doctree_dir,
	'linkcheck',
	warningiserror=False)
	app.build()

[michaelcdevin]

That's correct @ryancoe, the warningiserror=False line results in that behavior