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

Fix linting issues #25

Merged
merged 8 commits into from
Feb 8, 2022
Merged

Fix linting issues #25

merged 8 commits into from
Feb 8, 2022

Conversation

wwarriner
Copy link
Contributor

Fixed various linting issues. There is one known issue we can't resolve on the VSCode side of things. The VSCode markdown linter extension thinks that admonitions are indented code blocks, and it complains if there are both fenced (with backtick `) and indented code blocks in the same document. This can't currently be resolved.

@mdefende
Copy link
Member

mdefende commented Feb 8, 2022

There is a way to ignore specific linting warning codes if we want to investigate that. If we don't think there are going to be actual instances of this warning in what we are creating down the line

@mdefende
Copy link
Member

mdefende commented Feb 8, 2022

Additionally:

Individual warnings can be suppressed with comments in the Markdown file itself:

<!-- markdownlint-disable MD037 -->
deliberate space * in * emphasis
<!-- markdownlint-enable MD037 -->

Copy link
Member

@mdefende mdefende left a comment

Choose a reason for hiding this comment

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

Seems great

@mdefende mdefende merged commit 969ace8 into uabrc:main Feb 8, 2022
jgordini added a commit to jgordini/uabrc.github.io that referenced this pull request Oct 24, 2023
…onda-comments

[Sweep Rules] Update comments about Anaconda and Miniconda installation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants