Skip to content

chore: reafactoring makefile and packaging #430

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 32 commits into
base: main
Choose a base branch
from
Draft

chore: reafactoring makefile and packaging #430

wants to merge 32 commits into from

Conversation

@cdesiles
Copy link
Contributor

@cdesiles cdesiles commented Aug 12, 2024 •
edited
Loading

Disclaimer

This PR will be closed as soon as each item will be merged in smaller PRs.

See :

Previous work changes

  • Rely on php-scoper's phar instead of docker
  • Ignore php-scoper phar and MAC stuff
  • Update composer's lockfile
  • Updated the github actions workflow
  • PHPUnit works locally and in docker
  • PHPStan works locally and in docker
  • PHP CS Fixer fixed all source code (via PHP 8 ou PHP plus ancien ?)
  • E2E tests passed
  • Est-il possible d'empêcher l'upgrade du module lors de l'install ? Dans les init-scripts ? Un update en bdd en 99.99.99?
  • Keep the Herve's way of coding with TDD: the IDE should be plugged to the phpunit tests
  • Split PHP unit tests: unit testing and feature testing

@cdesiles cdesiles self-assigned this Aug 12, 2024
@cdesiles cdesiles changed the title fix/make packaging feat/reafactoring: makefile and packaging Aug 13, 2024
@cdesiles cdesiles changed the title feat/reafactoring: makefile and packaging feat: reafactoring makefile and packaging Aug 13, 2024
@cdesiles cdesiles changed the title feat: reafactoring makefile and packaging chore: reafactoring makefile and packaging Aug 13, 2024
github-advanced-security[bot]
github-advanced-security bot found potential problems Aug 27, 2024
View reviewed changes
@cdesiles cdesiles added enhancement New feature or request quality assurance needed Trigger continuous deployment integration fix and removed ready to review labels Aug 30, 2024
@sonarqubecloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
6 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

cdesiles
cdesiles commented Sep 4, 2024
View reviewed changes
Makefile
autoindex: ${TOOLS}
autoindex prestashop:add:index "${WORKDIR}"

# target: header-stamp - Add header stamp to files
Copy link
Contributor Author

Choose a reason for hiding this comment

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

.phony manquant

cdesiles
cdesiles commented Sep 4, 2024
View reviewed changes
Makefile
php-scoper: ${WORKDIR}/vendor ${WORKDIR}/vendor/.scoped

# target: autoindex - Automatically add index.php to each folder (fix for misconfigured servers)
autoindex: ${TOOLS}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

.phony manquant

cdesiles
cdesiles commented Sep 4, 2024
View reviewed changes
Makefile
header-stamp: ${TOOLS}
header-stamp --target="${WORKDIR}" --license="assets/afl.txt" --exclude=".github,node_modules,vendor,tests,_dev"

# target: version - Update the version in various files
Copy link
Contributor Author

Choose a reason for hiding this comment

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

.phony manquant

github-advanced-security[bot]
github-advanced-security bot found potential problems Oct 21, 2024
View reviewed changes
@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
1 Security Hotspot
E Security Rating on New Code (required ≥ A)

See analysis details on SonarCloud

Catch issues before they fail your Quality Gate with our IDE extension SonarLint

@cdesiles cdesiles added invalid This doesn't seem right WIP and removed quality assurance needed Trigger continuous deployment integration labels Jan 14, 2025
@hschoenenberger hschoenenberger marked this pull request as draft February 21, 2025 16:16
@sonarqubecloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
2 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hschoenenberger hschoenenberger Awaiting requested review from hschoenenberger

@antoinemetifeu antoinemetifeu Awaiting requested review from antoinemetifeu

Assignees

@cdesiles cdesiles

Labels

enhancement New feature or request fix invalid This doesn't seem right

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@cdesiles @antoinemetifeu @hschoenenberger