-
Notifications
You must be signed in to change notification settings - Fork 0
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
34 feature canviar text box per un botó #53
The head ref may contain hidden characters: "34-feature-canviar-text-box-per-un-bot\u00F3"
Conversation
The latest updates on TiDB Serverless Branch. Learn more about branching GitHub integration↗︎
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
M'ho estic mirant des del mòbil i no puc valorar el tema dels fitxers de yarn (però a simple vista només sembla una reordenació). La resta de canvis (els que importen) estan perfectes!
Si no vaig errat (rectifica'm @puigde) diria que només nosaltres dos podíem fer merge a master fins ara. També et faltava cambiar la PR de draft a PR PR, ja t'ho he fet, i al ferho ha sortit la opció de fer squash and merge. |
No se si parlem del mateix, hi ha una opció per "squashejar" commits, és a dir, quan es faci merge, que nms aparegui un sol commir (en comptes dels meus 3 + el merge). Però per fer-ho crec que heu d'activar la opció de poder fer squashos. |
Nono, ja està activada segur @marcfranquesa pq amb la resta de prs ho hem fet |
aaah val merci, que noob sorry |
@PauMatas @marcfranquesa el package-lock.json s'ha creat perquè s'haurà instal·lat alguna cosa utilitzant npm en comptes de yarn. Intentem tenir-ho tot únicament amb yarn en futures ocasions així ens estalviem possibles problemes. |
Ja que estava he ficat els dos butonets, he optat per color claret que ja està massa saturat del fosc.