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

[nightly] Fix markdown linting errors #5811

Merged
merged 2 commits into from
Aug 19, 2024
Merged

Conversation

lbussell
Copy link
Contributor

Resolves markdown linting error: README.md:3 MD036/no-emphasis-as-heading Emphasis used instead of a heading [Context: "IMPORTANT"]

@lbussell lbussell enabled auto-merge (squash) August 16, 2024 22:53
@lbussell lbussell changed the title Use better format for nightly repo warnings [nightly] Fix markdown linting errors Aug 16, 2024
**The images from the dotnet/nightly repositories include last-known-good (LKG) builds for the next release of [.NET](https://github.com/dotnet/core).**

**See [dotnet](https://hub.docker.com/r/microsoft/dotnet-aspire-dashboard/) for images with official releases of [.NET](https://github.com/dotnet/core).**
> **Important**: The images from the dotnet/nightly repositories include last-known-good (LKG) builds for the next release of [.NET](https://github.com/dotnet/core).
Copy link
Member

Choose a reason for hiding this comment

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

I know this is unrelated to your changes but I'm surprised the linter is ok with having text before the top-level header has even been defined.

Copy link
Member

Choose a reason for hiding this comment

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

That's because the . prefixed directories are not scanned. They break this rule which has a lot of value outside of this directory. I chose to leave them unscanned due to the overlap w/the GH readmes.

@lbussell lbussell merged commit 5530747 into dotnet:nightly Aug 19, 2024
111 checks passed
@MichaelSimons
Copy link
Member

I apologize for enabling the lint GH action before ensuring nightly was all fixed up. That was an oversight on my part.

lbussell added a commit to lbussell/dotnet-docker that referenced this pull request Sep 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants