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

[automated]: crowdin sync #7542

Closed
wants to merge 2 commits into from
Closed

[automated]: crowdin sync #7542

wants to merge 2 commits into from

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Mar 8, 2025

New Crowdin translations from the Node.js Crowdin project

@github-actions github-actions bot requested a review from a team as a code owner March 8, 2025 17:20
Copy link

vercel bot commented Mar 8, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
nodejs-org ✅ Ready (Inspect) Visit Preview Mar 8, 2025 5:23pm

Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Comment on lines +44 to +62
<tr>
<td>
<img alt="Logo Apilado Oscuro de Node.js" src="/static/logos/nodejsStackedDark.svg" className="h-[164px] w-[267px]" width="267" height="164" />
</td>

<td>
<img alt="Logo Apilado Claro de Node.js" src="/static/logos/nodejsStackedLight.svg" className="h-[164px] w-[267px] rounded bg-neutral-950 p-2 dark:bg-transparent" width="267" height="164" />
</td>
</tr>

<tr>
<td>
<img alt="Logo Apilado Negro de Node.js" src="/static/logos/nodejsStackedBlack.svg" />
</td>

<td>
<img alt="Logo Apilado Blanco de Node.js" src="/static/logos/nodejsStackedWhite.svg" className="rounded bg-neutral-950 p-2 dark:bg-transparent" />
</td>
</tr>
Copy link
Collaborator

Choose a reason for hiding this comment

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

this is the thrashing we are trying to avoid - as far as I can tell, its not coming from crowdin - i wrote a short script to do this locally - and confirmed input and output, so my current theory now is that we call format differently. going to try and verify locally again

Copy link
Collaborator

Choose a reason for hiding this comment

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

npm run lint on the local branch complains about the home pages of each locale - they have unique codeblocks - i will feed that into my program

Copy link
Collaborator

Choose a reason for hiding this comment

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

huh not only a lint - a failure
image

i bet the indentation is throwing off the markdown parsing

@bmuenzenmeyer bmuenzenmeyer added the github_actions:pull-request Trigger Pull Request Checks label Mar 8, 2025
@github-actions github-actions bot removed the github_actions:pull-request Trigger Pull Request Checks label Mar 8, 2025
Copy link
Contributor Author

github-actions bot commented Mar 8, 2025

Lighthouse Results

URL Performance Accessibility Best Practices SEO Report
/en 🟢 99 🟢 100 🟢 100 🟢 91 🔗
/en/about 🟢 99 🟢 100 🟢 100 🟢 91 🔗
/en/about/previous-releases 🟢 99 🟢 100 🟢 100 🟢 92 🔗
/en/download 🟠 89 🟢 100 🟢 100 🟢 91 🔗
/en/blog 🟢 100 🟢 100 🟢 96 🟢 92 🔗

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.

2 participants