florian.llimos/siahbva
1
0
Fork 0
Code Issues Pull Requests 2 Actions Packages Projects Releases Wiki Activity

feat: base of the project with npm dependencies #2

Open
florian.llimos wants to merge 3 commits from base-project into main
pull from: base-project
merge into: florian.llimos:main
Conversation 0 Commits 3 Files Changed 6 +279 -2
florian.llimos commented 2025-02-19 09:43:01 +01:00
Owner
Copy Link
No description provided.
florian.llimos added 1 commit 2025-02-19 09:43:02 +01:00
fabien.amarger requested changes 2025-02-20 09:27:04 +01:00
fabien.amarger left a comment
Collaborator
Copy Link

Le fichier style.css est la version CSS du less de uikit? Si oui on ne veut pas forcément le versionner puisqu'on peut le récupérer en transpilant le less de la dépendance, avec la commande npm run build-css.

De manière générale on essaie de versionner le minimum, tout ce qui peut être build, récupéré, etc, on le met pas dans git.
Le README.md est là pour expliquer les commandes à taper pour avoir un environnement fonctionnel en local.

Sinon c'est top!
C'est exactement ce que je voulais en terme de qualité et de processus.
Bravo 👏

Je met "demande des changements" pour enlever le style.css du versionnement, changer le href du style.css dans le index.html, et peut être mettre un readme pour expliquer la commande build-css. C'est pas grand chose mais c'est la mini étape avant d'avoir un truc nickel. (et si au passage tu peux me mettre le package-lock alors là c'est la cerise :p )

Le fichier `style.css` est la version CSS du less de uikit? Si oui on ne veut pas forcément le versionner puisqu'on peut le récupérer en transpilant le less de la dépendance, avec la commande `npm run build-css`. De manière générale on essaie de versionner le minimum, tout ce qui peut être build, récupéré, etc, on le met pas dans git. Le README.md est là pour expliquer les commandes à taper pour avoir un environnement fonctionnel en local. Sinon c'est top! C'est exactement ce que je voulais en terme de qualité et de processus. Bravo 👏 Je met "demande des changements" pour enlever le style.css du versionnement, changer le href du style.css dans le index.html, et peut être mettre un readme pour expliquer la commande build-css. C'est pas grand chose mais c'est la mini étape avant d'avoir un truc nickel. (et si au passage tu peux me mettre le package-lock alors là c'est la cerise :p )
.gitignore Outdated
@ -0,0 +1,3 @@
node_modules/
package-lock.json
fabien.amarger commented 2025-02-20 09:20:53 +01:00
Collaborator
Copy Link

lui on veut bien le versionner pour pouvoir faire npm ci et non npm install justement. La différence est que ci va installer les versions fixes des dépendances en regardant le package-lock.json alors que le install va essayer de mettre à jour certaines dépendances s'il y a des MAJ et va mettre à jour le package-lock.

Pour du dev on veut bien tester des mises à jours de dépendances, pour du déploiement en prod c'est bien de garder exactement le même environnement, d'où l'importance du package-lock dans le versionnement.

lui on veut bien le versionner pour pouvoir faire `npm ci` et non `npm install` justement. La différence est que `ci` va installer les versions fixes des dépendances en regardant le `package-lock.json` alors que le `install` va essayer de mettre à jour certaines dépendances s'il y a des MAJ et va mettre à jour le package-lock. Pour du dev on veut bien tester des mises à jours de dépendances, pour du déploiement en prod c'est bien de garder exactement le même environnement, d'où l'importance du package-lock dans le versionnement.
florian.llimos marked this conversation as resolved
index.html Outdated
@ -0,0 +5,4 @@
<meta name="viewport" content="width=device-width, initial-scale=1.0">
<title>Siahbva</title>
<link rel="icon" href="media/logo-couleur-2.svg">
<link rel="stylesheet" href="style.css">
fabien.amarger commented 2025-02-20 09:23:58 +01:00
Collaborator
Copy Link

Si je comprend bien, c'est plutôt dist/style.css pour le href non ?

Si je comprend bien, c'est plutôt `dist/style.css` pour le href non ?
florian.llimos marked this conversation as resolved
florian.llimos force-pushed base-project from 068e160c79 to 523d949503 2025-02-21 16:12:59 +01:00 Compare
florian.llimos added 1 commit 2025-02-21 16:15:26 +01:00
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions.

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin base-project:base-project
git checkout base-project
Sign in to join this conversation.
Reviewers
No reviewers
fabien.amarger
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: florian.llimos/siahbva#2
No description provided.
Delete Branch "base-project"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?