Skip to content

Fix Markdown checks #673

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

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Conversation

fandreuz
Copy link

@fandreuz fandreuz commented Jun 13, 2025

[...]
CHANGELOG.md:253 MD024/no-duplicate-heading Multiple headings with the same content [Context: "Removed"]
CHANGELOG.md:262 MD024/no-duplicate-heading Multiple headings with the same content [Context: "Added"]
CHANGELOG.md:272 MD024/no-duplicate-heading Multiple headings with the same content [Context: "Maturity"]
[...]
  • Disable MD059, the transgressions I see don't look that bad to me:
CONTRIBUTING.md:27:176 MD059/descriptive-link-text Link text should be descriptive [Context: "[here]"]

View meeting notes of previous SIG calls in this [google doc](https://docs.google.com/spreadsheets/d/1SYKfjYhZdm2Wh2Cl6KVQalKg_m4NhTPZqq-8SzEVO6s/edit?gid=0#gid=0) 
as stated [here](https://github.com/open-telemetry/community/?tab=readme-ov-file#governing-bodies) to stay up to date.

@fandreuz fandreuz requested a review from a team as a code owner June 13, 2025 23:28
@reyang
Copy link
Member

reyang commented Jun 13, 2025

Thanks @fandreuz! I see the package.json and package-lock.json being deleted in the PR, is this intentional?

@fandreuz
Copy link
Author

Thanks @fandreuz! I see the package.json and package-lock.json being deleted in the PR, is this intentional?

Hi @reyang, that's intentional because the npm dependencies are now embedded in a Docker container rather than being installed on the host. Let me know if that's a problem.

@arminru arminru requested a review from a team June 17, 2025 10:58
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.

4 participants