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.

Sign up for GitHub

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

Jump to bottom

Testing utility apis #237

Merged
merged 7 commits into from
Apr 28, 2021
Merged

Testing utility apis #237

merged 7 commits into from
Apr 28, 2021

Conversation

clarasdfgh
Copy link

@clarasdfgh clarasdfgh commented Oct 1, 2020 •
edited by AntonioCardenas
Loading

Lista de Verificación del PR

Comprueba si tu PR cumple los siguientes requisitos:

  • El mensaje de commit esta conforme con nuestras reglas
  • Probe los cambios que agregué (arreglo de bugs / funcionalidades)
  • Revisé previamente las traducciones o cambios de contenido
  • Consulté el diccionario de términos en español
  • He creado dos archivos con la extensión correspondiente(.en.md para el archivo en inglés y .md para el Archivo en español)
  • La PR fue enlazada con el issue correspondiente

Tipo de PR

¿Qué tipo de cambio introduce este PR?

  • Bugfix
  • Funcionalidad
  • Actualización de el estilo del código (formato, variables locales)
  • Refactorización (no cambios en la funcionalidad, no cambios en el api)
  • Cambios relacionados al build
  • Cambios relacionados al CI (Integración continua)
  • Cambios en el contenido de la documentación
  • Cambios en la aplicación / infraestructura de angular.io
  • Otro... Por favor describe la:

¿Cuál es el comportamiento actual?

¿Cuál es el nuevo comportamiento?

#fix #231

@CLAassistant
Copy link

CLAassistant commented Oct 1, 2020 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@clarasdfgh
Copy link
Author

Sí, consulté el diccionario. Cómo debería añadir el archivo en inglés, para distinguirlo del español?

@AntonioCardenas AntonioCardenas requested a review from a team October 1, 2020 16:17
@AntonioCardenas
Copy link

Muchas gracias @

Sí, consulté el diccionario. Cómo debería añadir el archivo en inglés, para distinguirlo del español?

Deberias de crear un nuevo archivo con el mismo nombre pero con la extension .en.md ejemplo: testing-utility-apis.en.md

@clarasdfgh
Copy link
Author

Añadido, lamento el despiste

@AntonioCardenas
Copy link

AntonioCardenas commented Oct 1, 2020 •
edited
Loading

Añadido, lamento el despiste

Muchisimas gracias no te preocupes a todos nos pasa de vez en cuando 😅 otra cosa no olvides agregar # y el númeroo de la issue para enlazar a la issue que estás resolviendo

@Splaktar
Copy link
Member

Splaktar commented Oct 2, 2020

@AntonioCardenas ¿Este PR no parece tener ningún cambio relacionado con el #223?

@AntonioCardenas
Copy link

AntonioCardenas commented Oct 2, 2020 •
edited by Splaktar
Loading

@AntonioCardenas ¿Este PR no parece tener ningún cambio relacionado con el #223?

Mi error , ya lo arregle @Splaktar

gustavguez
gustavguez requested changes Jan 20, 2021
View reviewed changes
aio/content/guide/testing-utility-apis.md Outdated Show resolved Hide resolved
aio/content/guide/testing-utility-apis.md Outdated Show resolved Hide resolved
Splaktar
Splaktar requested changes Jan 20, 2021
View reviewed changes
Copy link
Member

@Splaktar Splaktar left a comment

Choose a reason for hiding this comment

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

¡Gracias por su colaboración!

Por favor, rebase este PR.

aio/content/guide/testing-utility-apis.md Outdated Show resolved Hide resolved
aio/content/guide/testing-utility-apis.md Outdated Show resolved Hide resolved
@Splaktar @gustavguez
docs: fix typo and translate title
45a1311 
Co-authored-by: Gustavo Rodríguez <[email protected]>
@Splaktar
Copy link
Member

Parece que la commit de las sugerencias hizo que el CI rebase y ejecutara todas las pruebas. Ahora para ver si se vuelven verdes ...

gustavguez
gustavguez previously requested changes Apr 22, 2021
View reviewed changes
aio/content/guide/testing-utility-apis.md Outdated Show resolved Hide resolved
aio/content/guide/testing-utility-apis.md Outdated Show resolved Hide resolved
aio/content/guide/testing-utility-apis.md Outdated Show resolved Hide resolved
@gustavguez
Copy link
Member

@clarasdfgh disculpa la demora, estamos teniendo problemas con los commits messages, para poder mergear:

Examining 7 commit(s) in the provided range: 3d084ac9104a756269af2b5969963369e0f9aaa6..a2c6f29e5a05b628a2bbe4a90cc48631553fa8cc
INVALID COMMIT MSG: 
────────────────────────────────────────
Delete testing-utility-apis.md
────────────────────────────────────────
ERROR: 
  The commit message header does not match the expected format.

The expected format for a commit is: 
<type>(<scope>): <subject>

<body>

Exited with code exit status 1
CircleCI received exit code 1

Podrás cambiar los commits messages para que cumplan y pasen los tests?
Saludos!

@Splaktar Splaktar merged commit a2aa45d into angular-hispano:master Apr 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Splaktar Splaktar Splaktar approved these changes

@gustavguez gustavguez gustavguez left review comments

Assignees

@clarasdfgh clarasdfgh

Labels
hacktoberfest-accepted
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Traducir: guide/testing-utility-apis.md
5 participants
@clarasdfgh @CLAassistant @AntonioCardenas @Splaktar @gustavguez