-
Notifications
You must be signed in to change notification settings - Fork 3
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
EPIC README + publishing marketplace #110
Conversation
7e07642
to
2072b4d
Compare
Welcome to Codecov 🎉Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment. Thanks for integrating Codecov - We've got you covered ☂️ |
Para el #49, estoy usando esta Github Actions que te
con eso se simplifica bastante la creación del tag nuevo: igualmente hay pasos manuales que son necesarios. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maravilloso todoooo!!!! 🌟
Alta magia tiraste 🪄
@@ -6,7 +6,6 @@ | |||
.gitignore | |||
**/tsconfig.json | |||
**/tsconfig.base.json | |||
**/node_modules/** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mmmmm esto qué hace? Estamos seguros que no lo queremos?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
eso es lo que rompió la 0.1.1, node_modules no lo podemos ignorar porque no te trae las dependencias de vscode
Ahora que tenemos publicado el componente en el Marketplace, hay que pulir algunas cositas que son cosas que pueden atraer potenciales devs o ayudar a gente que quiera chusmear Wollok:
Una vez mergeado a master hay que: