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

Add Sign Language Video Style Guide (SPA) to content/es/style-guide/index.md #3357

Merged
merged 5 commits into from
Nov 22, 2024

Conversation

tech-andgar
Copy link

Describe your changes

added the sign language video style guide (lang/es)

Related issue number or link (ex: resolves #issue-number)

related to issue #3044 (comment)

Checklist before opening this PR (put x in the checkboxes)

  • This PR does not contain plagiarism
    • don’t copy other people’s work unless you are quoting and contributing it to them.
  • I have signed off on all commits
    • signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

Copy link

netlify bot commented Nov 20, 2024

Deploy Preview for cncfglossary ready!

Name Link
🔨 Latest commit a1d95e8
🔍 Latest deploy log https://app.netlify.com/sites/cncfglossary/deploys/6740bbac40ce240008e9f637
😎 Deploy Preview https://deploy-preview-3357--cncfglossary.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@tech-andgar tech-andgar changed the title added the sign language video style guide (lang/es) Add Sign Language Video Style Guide (SPA) to content/es/style-guide/index.md Nov 20, 2024
@tech-andgar
Copy link
Author

Based of #3313

Ping: @seokho-son

Copy link
Collaborator

@electrocucaracha electrocucaracha left a comment

Choose a reason for hiding this comment

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

@tech-andgar muchas gracias por la actualización en el documento, parece ser que algunas palabras no están registradas en el diccionario local, podrías agregarlas en orden alfabético para que pueda pasar el CI?

content/es/style-guide/_index.md Outdated Show resolved Hide resolved
content/es/style-guide/_index.md Show resolved Hide resolved
Co-authored-by: Victor Morales <[email protected]>
Signed-off-by: Andres Garcia <[email protected]>
@tech-andgar
Copy link
Author

tech-andgar commented Nov 20, 2024

@tech-andgar muchas gracias por la actualización en el documento, parece ser que algunas palabras no están registradas en el diccionario local, podrías agregarlas en orden alfabético para que pueda pasar el CI?

@electrocucaracha
Este diccionario local se actualiza en orden alfabético.

@electrocucaracha
Copy link
Collaborator

@tech-andgar las palabras que agregaste están en el orden correcto, pero parece ser que tu editor de texto está cambiando, el fin del documento, por lo general eso sucede con editores windows o por alguna configuración en git (git config --global core.autocrlf true).

Por otra parte, recuerda que cada commit debe ser firmado, en caso contrario no podra pasar el verificador de DCO

Signed-off-by: Andrés García <[email protected]>
@tech-andgar
Copy link
Author

@tech-andgar las palabras que agregaste están en el orden correcto, pero parece ser que tu editor de texto está cambiando, el fin del documento, por lo general eso sucede con editores windows o por alguna configuración en git (git config --global core.autocrlf true).

Por otra parte, recuerda que cada commit debe ser firmado, en caso contrario no podra pasar el verificador de DCO

@electrocucaracha
Gracias por recordar este punto y ya lo arreglé.

@electrocucaracha
Copy link
Collaborator

De mi parte video debería no tener acento y ser agregado al .wordlist, pero me gustaría conocer lo que opinan @ramrodo @krol3 @jossemarGT @92nqb

@ramrodo
Copy link
Collaborator

ramrodo commented Nov 21, 2024

De mi parte video debería no tener acento y ser agregado al .wordlist, pero me gustaría conocer lo que opinan @ramrodo @krol3 @jossemarGT @92nqb

Yo también estoy acostumbrado a escribir "video" sin acento. Sin embargo, investigando en la RAE, ambas son válidas: la opción sin acento mayormente usada en América Latina y con acento usada en España: https://www.rae.es/dpd/video

Mi conclusión personal sería igual que @electrocucaracha, usar la opción sin acento (y agregarla al .wordlist) ya que es la opción más usada en diferentes países de habla hispana.

@krol3
Copy link
Collaborator

krol3 commented Nov 22, 2024

De mi parte video debería no tener acento y ser agregado al .wordlist, pero me gustaría conocer lo que opinan @ramrodo @krol3 @jossemarGT @92nqb

Yo también estoy acostumbrado a escribir "video" sin acento. Sin embargo, investigando en la RAE, ambas son válidas: la opción sin acento mayormente usada en América Latina y con acento usada en España: https://www.rae.es/dpd/video

Mi conclusión personal sería igual que @electrocucaracha, usar la opción sin acento (y agregarla al .wordlist) ya que es la opción más usada en diferentes países de habla hispana.

+1 sin acento video

@electrocucaracha
Copy link
Collaborator

@tech-andgar puedes agregar la palabra video al wordlist para que pase el CI y aprobar este cambio?

Signed-off-by: Andrés García <[email protected]>
@tech-andgar
Copy link
Author

@tech-andgar puedes agregar la palabra video al wordlist para que pase el CI y aprobar este cambio?

@electrocucaracha
Listo

@ramrodo ramrodo merged commit 363571d into cncf:dev-es Nov 22, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lang/es for Spanish
Projects
Development

Successfully merging this pull request may close these issues.

4 participants