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

project-structure #1

Open
florian.llimos wants to merge 4 commits from project-structure into main
pull from: project-structure
merge into: florian.llimos:main
Conversation 1 Commits 4 Files Changed 6 +45 -2
florian.llimos commented 2025-02-11 14:43:11 +01:00
Owner
Copy Link
No description provided.
florian.llimos added 4 commits 2025-02-11 14:43:11 +01:00
fabien.amarger requested changes 2025-02-11 15:22:47 +01:00
fabien.amarger left a comment
Collaborator
Copy Link

Je n'aurais intégré les images que lorsqu'elles sont utilisées dans le HTML

Je n'aurais intégré les images que lorsqu'elles sont utilisées dans le HTML
.gitignore
@ -0,0 +1,2 @@
css/
fabien.amarger commented 2025-02-11 15:18:05 +01:00
Collaborator
Copy Link

Tu ignores les dossiers CSS et JS du dépôt ? ça ne fait pas partie des rendus du site ? ou alors, je n'ai pas compris pourquoi c'est là ça.

Tu ignores les dossiers CSS et JS du dépôt ? ça ne fait pas partie des rendus du site ? ou alors, je n'ai pas compris pourquoi c'est là ça.
florian.llimos commented 2025-02-11 16:02:45 +01:00
Author
Owner
Copy Link

J'ai ignoré les dossiers css & js car vu que ce sont des dépendances, je ne vois pas trop l'utilité de les push sur le dépôt alors qu'on peut les récupérer au besoin sur le site du framework directement.

J'ai ignoré les dossiers css & js car vu que ce sont des dépendances, je ne vois pas trop l'utilité de les push sur le dépôt alors qu'on peut les récupérer au besoin sur le site du framework directement.
fabien.amarger commented 2025-02-11 18:04:42 +01:00
Collaborator
Copy Link

En utilisant npm et la configuration avec le package.json (https://docs.npmjs.com/cli/v10/configuring-npm/package-json?v=true) il ne sera plus nécessaire d'ignorer ces dossiers dans le versionnement, mais par contre, il faudra ignorer le dossier node_modules comme il est d'usage sur les projets Web de ce type.

Ça me semble plus intéressant de partir sur ce type de gestion de dépendances pour éviter d'avoir à installer manuellement une dépendance, et permettre de gérer facilement ces dépendances, notamment dans un système de gestion de version tel que Git.

En utilisant npm et la configuration avec le package.json (https://docs.npmjs.com/cli/v10/configuring-npm/package-json?v=true) il ne sera plus nécessaire d'ignorer ces dossiers dans le versionnement, mais par contre, il faudra ignorer le dossier `node_modules` comme il est d'usage sur les projets Web de ce type. Ça me semble plus intéressant de partir sur ce type de gestion de dépendances pour éviter d'avoir à installer manuellement une dépendance, et permettre de gérer facilement ces dépendances, notamment dans un système de gestion de version tel que Git.
README.md
@ -4,0 +6,4 @@
## ⚙️ Étape 2 - Télécharger les dépendances :
Ouvrez sur ce lien **https://getuikit.com/docs/introduction** et téléchargez Uikit.
fabien.amarger commented 2025-02-11 15:19:53 +01:00
Collaborator
Copy Link

UIKit est disponible sur npmjs (c.f. https://www.npmjs.com/package/uikit ) je pense que ça serait mieux de versionner le package.json et le package-lock.json avec la dépendance dedans plutôt que de le télécharger et de l'installer manuellement.

UIKit est disponible sur npmjs (c.f. https://www.npmjs.com/package/uikit ) je pense que ça serait mieux de versionner le package.json et le package-lock.json avec la dépendance dedans plutôt que de le télécharger et de l'installer manuellement.
florian.llimos commented 2025-02-11 16:03:16 +01:00
Author
Owner
Copy Link

Je ne connaissais pas cette façon de faire.

Je ne connaissais pas cette façon de faire.
fabien.amarger commented 2025-02-11 18:04:55 +01:00
Collaborator
Copy Link

c.f. mon commentaire précédent.

c.f. mon commentaire précédent.
florian.llimos commented 2025-02-11 16:04:36 +01:00
Author
Owner
Copy Link

Je n'aurais intégré les images que lorsqu'elles sont utilisées dans le HTML

J'ai mis toutes les images directement dès que le début car l'objectif de ma branche et de tout avoir créé au début pour pouvoir travailler en ayant tout depuis cette branche. Tout sauf les dépendances.

> Je n'aurais intégré les images que lorsqu'elles sont utilisées dans le HTML J'ai mis toutes les images directement dès que le début car l'objectif de ma branche et de tout avoir créé au début pour pouvoir travailler en ayant tout depuis cette branche. Tout sauf les dépendances.
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 project-structure:project-structure
git checkout project-structure
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#1
No description provided.
Delete Branch "project-structure"

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?