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

Tweak documentation verification tests #1504

Merged
merged 1 commit into from
Nov 4, 2022

Conversation

Gedochao
Copy link
Contributor

@Gedochao Gedochao commented Oct 27, 2022

  • fixed outdated instructions for running documentation verification tests in the readme
  • the tests should now also be run on guides, which seem to have been skipped
  • fixed a few guides documents to make them pass tests

@Gedochao Gedochao marked this pull request as ready for review October 28, 2022 08:22
@Gedochao Gedochao changed the title Tweak documentation verification tests with Sclicheck Tweak documentation verification tests Oct 28, 2022
@Gedochao Gedochao force-pushed the tweak-sclicheck-docs-tests branch 2 times, most recently from a9e8bd8 to 0741459 Compare November 2, 2022 08:22
@Gedochao Gedochao force-pushed the tweak-sclicheck-docs-tests branch 2 times, most recently from 8866c07 to b2e3a93 Compare November 2, 2022 09:11
@Gedochao Gedochao added documentation Improvements or additions to documentation internal Internal or build-related changes labels Nov 3, 2022
README.md Show resolved Hide resolved
website/docs/guides/configuration.md Show resolved Hide resolved
  - update `README.md`
  - run docs verification tests on guides as well
  - fix some guides to prevent test failures on them (`compile`, `configuration`, `dependencies` and `scripts`)
@Gedochao Gedochao merged commit bb3fca7 into VirtusLab:main Nov 4, 2022
@Gedochao Gedochao deleted the tweak-sclicheck-docs-tests branch November 4, 2022 13:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation internal Internal or build-related changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants