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

WIP: Mis à jour du projet #10

Open
wants to merge 17 commits into
base: master
Choose a base branch
from
Open

WIP: Mis à jour du projet #10

wants to merge 17 commits into from

Conversation

rparpa
Copy link
Collaborator

@rparpa rparpa commented Feb 2, 2020

  • Màj des images utilisées (montée de version + full debian) ;
  • PHP 7.4 ;
  • Simplification de l'autoloader ;
  • Pattern singleton sur la connexion DB.

Même si le pattern singleton est parfois considéré deprecated, il simplifie pas mal l'utilisation de la connexion PDO et ça peut donner de bonnes discussions sur l'utilisation d'un autre pattern. WDYT ?

Quoi que au final ce n'est pas vraiment un singleton non plus, il est moins limitant

Rémi Parpaillon added 2 commits February 2, 2020 11:15
* Use debian images instead of alpine ;
* Upgrade to latest nginx / postgres / php.
* Upgrade twitter bootstrap 3 -> 4;
* Upgrade phpunit;
* Simplify db init;
* Simplify autoload.
@rparpa rparpa requested a review from Kirouane February 2, 2020 10:22
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.

2 participants