-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Formato del título de la PR #11
Comments
Yo tambien estaba en duda con eso, pero creo que el titulo de las PR da igual puedes usar el que te pone por defecto, pero lo importante es que tu solucion la pongas con la estructura de directorios correcta que se explica en la web
Creo que es asi, pero bueno sino que nos lo deje claro Moure. |
Sí, eso es lo que al final entiendo también. Aunque, en mi opinión, creo que sería mejor si se especificara un formato para el título de las PRs. Creo que se vería un poco más organizado. Por ejemplo, si ves ahora la sección de PRs y ves cada quien con su formato de título. |
Pues si, toda la razon, se veria mucho mas limpio, pero bueno de momento lo importante es la estructura. Mientras Moure no diga lo contrario pues habra todo tipo de titulos xD |
Comparto tu opinión @lombervid si mantenemos una convención sera mas fácil a la hora de "husmear" otras alternativas o lenguajes. |
si no quizás seguir la convención de Reto #[Numero del reto] [Lenguaje] , al final abajo en la PR ya pone el usuario. |
Bueno yo estoy de acuerdo tambien, podemos hacerlo nosotros asi y a ver si el resto de la gente lo coge como buena practica. |
Es mi primera vez en GitHub y no veo como modificar eso que comentáis. En mi fork si que tengo el archivo JS metido en un Path tal y como pide mouredev, pero al lanzar la PR me lo coloca tal cual, solo nombre de archivo. Funcionalidades no faltaran, pero de "intuitivo" no va sobrado esto que digamos. Saludos!, |
Cuando le das crear la PR, ahí puedes cambiar el título a cómo tú lo quieras (te pone uno por defecto pero puedes cambiarlo). También, una vez creado, puedes:
Ya con eso se debería de haber actualizado el título |
Oye, pues mil gracias por la ayuda!!. Saludos!; |
Hola a todo el mundo. No he establecido un formato para el título de la PR. Creo que sería suficiente con el número del reto y el lenguaje. De todas formas es algo que podría definir para el segundo reto. Muchas gracias por todo. |
Uso solo el método de strings, SPLIT(). Con ella voy haciendo trocitos para quedarme solo con los parámetros y sus valores.
## Describe tus cambios ***(Opcional)*** *Sobre todo aconsejable si la "Pull Request" se corresponde con una corrección adicional y no con la presentación de un ejercicio.* ## Comprobaciones Asegúrate de cumplir los siguientes puntos antes de realizar la "Pull Request": - [ ] El título de mi Pull Request sigue este formato: "Reto #[número] - [lenguaje_utilizado]". *(Ej: Reto #0 - Kotlin")* - [ ] El nombre el fichero que se corresponde con el de mi usuario en GitHub más la extensión del lenguaje. *(Ej: mouredev.kt)* - [ ] El fichero de corrección se encuentra dentro del directorio del ejercicio y en una carpeta con el nombre del lenguaje de programación utilizado en minúsculas. *(Ej: Reto #0/kotlin/mouredev.kt)* - [ ] He revisado que el nombre del directorio del lenguaje no es conflictivo: - c#, no csharp - c++, no cplusplus - go, no golang - javascript, no js - [ ] Únicamente he incluído los ficheros de ejercicios. No se aceptarán Pull Requests que contengan archivos adicionales asociados a editores de código o semejantes. ## Información * Tienes toda la información sobre los retos semanales en [retosdeprogramacion.com/semanales2023](https://retosdeprogramacion.com/semanales2023). * Cada semana se realizará la corrección en directo y publicación de un nuevo reto en [twitch.tv/mouredev](https://twitch.tv/mouredev). * Recuerda que tienes un grupo de apoyo llamado "reto-semanal" en [Discord](https://discord.gg/mouredev).
Hola.
Una cosa que no me termina de quedar claro es si ¿hay algún formato a seguir para el título de las PRs?
Por ejemplo, la primera la mandé con formato:
Reto #[NÚMERO-RETO]/[LENGUAJE-UTILIZADO]/[USUARIO].[EXTENSIÓN]
, ya que por está parte me hizo pensar que ese debía de ser el formato del título:Pero en esta última que hice ya me salía por defecto un formato
Solución Reto #[NÚMERO-RETO] [LENGUAJE-UTILIZADO]
Entonces, ¿este último es el formato que se debe de seguir?, ¿hay algún otro formato?, o ¿no importa el formato del título?
Un saludo.
The text was updated successfully, but these errors were encountered: