Skip to content

Conversation

@mikeesto
Copy link
Member

@mikeesto mikeesto commented Sep 2, 2025

Description

Corrects the markdown link formatting in the French translation. Seems like this happened in the recent sync.

Check List

  • I have read the Contributing Guidelines and made commit messages that follow the guideline.
  • I have run pnpm format to ensure the code follows the style guide.
  • I have run pnpm test to check if all tests are passing.
  • I have run pnpm build to check if the website builds without errors.
  • I've covered new added functionality with unit tests if necessary.

Copilot AI review requested due to automatic review settings September 2, 2025 06:30
@mikeesto mikeesto requested a review from a team as a code owner September 2, 2025 06:30
@vercel
Copy link

vercel bot commented Sep 2, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Updated (UTC)
nodejs-org Ready Ready Preview Sep 2, 2025 6:31am

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Fixes markdown link formatting in the French security reporting documentation by correcting malformed link syntax that caused rendering issues.

  • Corrects malformed markdown link syntax that had an escaped bracket
  • Ensures proper link rendering and resolves prettier formatting complaints

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@github-actions
Copy link
Contributor

github-actions bot commented Sep 2, 2025

Note

Your Pull Request seems to be updating Translations of the Node.js Website.

Whilst we appreciate your intent; Any Translation update should be done through our Crowdin Project.
We recommend giving a read on our Translation Guidelines.

Thank you!

@codecov
Copy link

codecov bot commented Sep 2, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.56%. Comparing base (fd6b7e6) to head (08605d8).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8135      +/-   ##
==========================================
- Coverage   76.60%   76.56%   -0.05%     
==========================================
  Files         115      115              
  Lines        9595     9595              
  Branches      321      322       +1     
==========================================
- Hits         7350     7346       -4     
- Misses       2244     2248       +4     
  Partials        1        1              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@avivkeller avivkeller added the fast-track Fast Tracking PRs label Sep 2, 2025
@avivkeller avivkeller added this pull request to the merge queue Sep 2, 2025
Merged via the queue into nodejs:main with commit 6c6c1da Sep 2, 2025
17 checks passed
@bmuenzenmeyer
Copy link
Contributor

@avivkeller - next time, you need someone else to approve your fast track request

@avivkeller
Copy link
Member

According to our guide:

Must have at least one 👍 reaction if the fast-tracker is the PR author

I am not the PR author, so I don't need a 👍, correct?

@mikeesto mikeesto deleted the fix-fr-mdx-link branch September 2, 2025 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fast-track Fast Tracking PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants