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

refactor (dist) moved templates to dist no need to copy #134

Closed
wants to merge 3 commits into from

Conversation

ptesei
Copy link
Contributor

@ptesei ptesei commented Jul 26, 2020

assets directory is deleted and regenerated as usual

assets directory is deleted and regenerated as usual
@firestar300
Copy link
Contributor

J'aurais fait le contraire : laisser les inté dans src/templates et les supprimer de dist.

Copy link
Contributor

@francoistibo francoistibo left a comment

Choose a reason for hiding this comment

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

oui même remarque que Milan. ca reste des fichiers sources donc ca a plus de sens qu'ils soient dans src/.

@asadowski10
Copy link
Member

Cela veut dire que la recette front ( GDP ) devra se faire dans /src/ si on retire de /dist/ ? et que les dev désormais devront aller récup les inté dans /src/ that's right ?

@francoistibo
Copy link
Contributor

Cela veut dire que la recette front ( GDP ) devra se faire dans /src/ si on retire de /dist/ ? et que les dev désormais devront aller récup les inté dans /src/ that's right ?

Oui (@ptesei c'est ca ?). Si ca n'implique rien d'autre que d'aller chercher ces fichiers dans src/ au lieu de dist/ pour la recette et les dev.

@ptesei
Copy link
Contributor Author

ptesei commented Jul 27, 2020

oui moi aussi j'avais pensé comme toi @firestar300 mais de cette manière on as dist que contiens juste assets
et c'est pa jolie. donc il faudrait bouger assets à la racine.
enfin les template sont e quelque sorte des "dist" désormais. et comme ça on change pas non habitudes ;)
Après s on peut choisir de le mettre ou on veut.

@firestar300
Copy link
Contributor

@firestar300 ouais je comprends ton point de vue mais pour moi ça ne doit pas faire partie de dist car ça ne sert pas le thème WordPress. Autant consacrer le dossier dist aux assets qui sont utilisés sur le thème WP à savoir le CSS, JS, les images statics, les SVG etc...

@firestar300
Copy link
Contributor

Donc voici ma vision des choses. On a un dossier :

src

qui contient toutes les sources qui se transformés de près ou de loin par Webpack.

  • le SCSS est transpilé en CSS et passe par moultes outils comme Stylelint ou encore PostCSS.
  • le JS est transpilé par Babel pour une rétrocompatibilité et passe par Eslint pour le linter
  • conf-img est un cas particulier car il ne subit pas de changement par Webpack et est spécifique au plugin Advanced Responsive Images

public

C'est le dossier qui contient toutes les sources qui seront copiés dans le dossier de build dist :

  • images
  • webfonts
  • templates PHP (intégration)

dist

C'est le dossier package qui sert à distribuer tout ce qui provient de public et de src en version compilé.

cd dist/ && php -S localhost:${PORT}
Copy link
Contributor

Choose a reason for hiding this comment

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

@ptesei pour qu'on aille sur localhost:3000 à la place de localhost:3000/dist/index.php

@firestar300
Copy link
Contributor

Je close cette PR. Elle est trop vieille et compliqué à rebase.

@francoistibo francoistibo deleted the refactor/dist-no-duplicate branch September 7, 2021 09:50
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.

None yet

4 participants