Codecov Report

Merging #2620 (18ec497) into main (e881d95) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #2620   +/-   ##
=======================================
  Coverage   68.83%   68.83%           
=======================================
  Files         119      119           
  Lines        1325     1325           
  Branches      333      333           
=======================================
  Hits          912      912           
  Misses        379      379           
  Partials       34       34           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

I'm seeing errors in your builds, don't forget to put your text on the YAML surrounded by quotes. Eg.:

description: "My description here"

Text usually needs to be quoted if contains special characters.

@AugustinMauroy looks great! I think we're finally close to get this merged.

Just a final review from @nodejs/nodejs-fr 🙏

Regarding these typos I mentioned, double-check the English version of your files 😄 (Just to ensure the title field is correct).

@AugustinMauroy I'm still waiting for those changes :)

I'm back from vacation. I am correcting all the mistakes on the pr.

@ovflowd

Looks good to me! @nodejs/nodejs-fr could you folks double-check here?

@AugustinMauroy could you delete the trademark markdown file? We removed that page from the website 🙇

After this I think we're good to merge :)

@ovflowd trademark was remove

I was going to merge this but github is telling me a conflict exists but doesn't list which file.

@benhalverson its strange ?

@benhalverson its strange ?

Yes

@benhalverson normally we can merge this pr