- Notifications
You must be signed in to change notification settings - Fork396
remove the requirement to sort the dict file.#360
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
… in lot of PRs. And adding a check for duplicated entries
Qué bueno!!! |
Está bueno esto. Aunque tengo unas preguntas: No entiendo porqué no habría conflictos si todos agregan palabras al final, cuando intente hacer merge va a notar que ambos modificaron la última línea y habrá conflicto igualmente, no? Por otro lado, quizás no es buena idea hacer que el build falle por palabras duplicadas y bloquee el PR, ya que igualmente son ignoradas. ¿Que opinás? |
Debido a que hemos sort del archivo del diccionario con el pre-commit esto esta creando un montón de conflictos a la hora de hacer merge de un nuevo PR.
Sí unicamente agregamos palabras nuevas al final del archivo esos conflictos van a producirse muy pocas veces.