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

Mejoras en la redacción del README #252

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

foopsss
Copy link

@foopsss foopsss commented Mar 1, 2024

Buenas noches. Me crucé con este proyecto de casualidad y noté que la redacción de varias oraciones del README no era del todo buena, así que decidí cambiarlo un poco para que se lea más natural. También arreglé un enlace que estaba roto y cambié un toque las instrucciones de instalación que giran en torno a Snap, porque no es necesario iniciar un nuevo proceso de Bash para correr esos comandos, además de que esas instrucciones asumen que todos los usuarios de Linux usan Bash, lo que no es cierto (existen otros procesadores de comandos como Zsh o Fish).

En fin, me gustaría que alguien le pegue una leída adicional al nuevo README y me diga si le parece bien. Leí las secciones que cambié varias veces y se me hace que se lee mucho mejor ahora.

* Cambios en la redacción de varias oraciones para que suenen más naturales.
* Remoción de un espacio blanco innecesario en un hiperenlace.
* Arreglo de un link a la sección "Issues".
Copy link

netlify bot commented Mar 1, 2024

Deploy Preview for weektodo-app ready!

Name Link
🔨 Latest commit 340a3d0
🔍 Latest deploy log https://app.netlify.com/sites/weektodo-app/deploys/65e14fc78d139000086c280b
😎 Deploy Preview https://deploy-preview-252--weektodo-app.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.

README.md Outdated

## Contributing

Weektodo is open-source. Pull requests and contributions are welcome! There are three ways to contribute: grab a [bug report](https://github.com/manuelernestog/issues?q=is%3Aopen+is%3Aissue+label%3Abug) or [feature suggestion](https://github.com/manuelernestog/issues?q=is%3Aissue+is%3Aopen+label%3Afeature) that has been marked `accepted` and dig in.
WeekToDo is open-source. Pull requests and contributions are welcome! There are three ways to contribute: fill a [bug report](https://github.com/manuelernestog/issues?q=is%3Aopen+is%3Aissue+label%3Abug), [suggest a feature](https://github.com/manuelernestog/weektodo/issues?q=is%3Aissue+is%3Aopen+label%3Afeature) or search for feature request that have been marked as `accepted` and dig in.
Copy link
Author

@foopsss foopsss Mar 1, 2024

Choose a reason for hiding this comment

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

Hago un punto aparte en este cambio porque acá traté de "adivinar" cual era la intención del mensaje original, que habla de tres maneras de contribuir aunque en la práctica solo se mencionan dos. ¿Está bien así?

EDIT: no me di cuenta e inicié una reseña en el primer cambio, que aparece marcado como "desactualizado" porque luego hice otros cambios (entre ellos uno donde añadí la "s" que faltaba en "search for feature request"), aunque este sigue estando correcto.

@foopsss
Copy link
Author

foopsss commented Mar 1, 2024

Aprovecho para comentar otra cuestión algo relacionada a #171. Ya es bastante evidente hoy por hoy que Snap perdió por goleada la pulseada de las tiendas de aplicaciones agnósticas, y hay distribuciones donde instalar Snap es un quilombo (como Fedora Silverblue y demás variantes inmutables, véase coreos/rpm-ostree#1711).

Mi opinión personal es que Snap debería ser dejado de lado en favor de Flatpak, que es el método de distribución favorecido por el grueso de distribuciones GNU/Linux y la comunidad en su mayoría. Esto no deja de ser mi opinión claro, jamás intenté empaquetar programas para Flatpak pero sé que para algunas cosas es un embole, mientras que para otras funciona de 10.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant