Skip to content

Conversation

mlunadia
Copy link
Contributor

[es] new translation - docs/concepts/sampling/index

@theletterf
Copy link
Member

theletterf commented Oct 23, 2025

@mlunadia Seems like the PR got lots of unrelated changes in — those are drift markers produced by npm run fix:i18n:status. @svrnm @vitorvasc Would we be OK with merging those in the same PR?

Otherwise, best if you'd stash your changes, rebase, pop the stash, and push again to your branch.

@vitorvasc
Copy link
Member

@mlunadia Seems like the PR got lots of unrelated changes in — those are drift markers produced by npm run fix:i18n:status. @svrnm @vitorvasc Would we be OK with merging those in the same PR?

Otherwise, best if you'd stash your changes, rebase, pop the stash, and push again to your branch.

The last time we ran these updates, we did them in separate PRs, one per locale, which also led to the creation of separate issues to track the updates. As an example, here are the last updates for es, zh, and ja.

I'd say we're fine to update the drift markers once again, but I'd do it in separate PRs and make sure to either create new issues to track the progress or update the existing ones.

@mlunadia
Copy link
Contributor Author

Thanks for the heads up @vitorvasc
I'll get on fixing it :)

@mlunadia
Copy link
Contributor Author

Replaced with #8200 to fix issues

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants