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

docs: translate to Spanish declare-dependencies.md #336

Merged
merged 5 commits into from
Oct 6, 2024

Conversation

RobPasMue
Copy link
Contributor

Related to #313

@willingc willingc added the translations Translation and localization of content label Aug 1, 2024
@flpm flpm added the lang-ES Spanish Translation label Aug 11, 2024
Copy link
Member

@flpm flpm left a comment

Choose a reason for hiding this comment

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

Thanks for another PR!

I found a bunch of mistakes in the original English guide while reviewing this PR, I fixed them in the translation here, but have a separate PR to fix them in the original English.

locales/es/LC_MESSAGES/package-structure-code.po Outdated Show resolved Hide resolved
locales/es/LC_MESSAGES/package-structure-code.po Outdated Show resolved Hide resolved
locales/es/LC_MESSAGES/package-structure-code.po Outdated Show resolved Hide resolved
locales/es/LC_MESSAGES/package-structure-code.po Outdated Show resolved Hide resolved
locales/es/LC_MESSAGES/package-structure-code.po Outdated Show resolved Hide resolved
locales/es/LC_MESSAGES/package-structure-code.po Outdated Show resolved Hide resolved
locales/es/LC_MESSAGES/package-structure-code.po Outdated Show resolved Hide resolved
locales/es/LC_MESSAGES/package-structure-code.po Outdated Show resolved Hide resolved
locales/es/LC_MESSAGES/package-structure-code.po Outdated Show resolved Hide resolved
locales/es/LC_MESSAGES/package-structure-code.po Outdated Show resolved Hide resolved
@RobPasMue
Copy link
Contributor Author

Thanks a lot (as always) for your thorough review @flpm!

Copy link
Member

@flpm flpm left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks @RobPasMue

@flpm flpm merged commit 551e6af into pyOpenSci:main Oct 6, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lang-ES Spanish Translation translations Translation and localization of content
Projects
Development

Successfully merging this pull request may close these issues.

3 participants