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

Initial boilerplate #1

Merged
merged 5 commits into from
Mar 18, 2024
Merged

Initial boilerplate #1

merged 5 commits into from
Mar 18, 2024

Conversation

asgrim
Copy link
Collaborator

@asgrim asgrim commented Feb 29, 2024

Some assumptions/opinions made for the development of the tool are:

  • Minimum PHP version 8.1, given that is the only PHP version in security support (or greater)
  • PHPUnit for unit testing; fairly strongly opinionated on this, but again, is a preference.
  • Psalm for static analysis; not strongly opinionated here, but I have more experience working with Psalm, so this is my preference
  • doctrine/coding-standard for CS; I feel this CS is comprehensive and fixes most code in a predictable way, and I am most familiar with phpcs (over say, php-cs-fixer).
  • I have used laminas/workflow-continuous-integration to run the CI workflow, as it is ludicrously easy to get up and running. Should we need to expand out of these realms in the future for static analysis/CS/linting/testing etc., we can of course

@asgrim asgrim added the enhancement New feature or request label Feb 29, 2024
composer.json Show resolved Hide resolved
@asgrim asgrim force-pushed the initial-boilerplate branch from d3e55e4 to 1342044 Compare February 29, 2024 08:31
@asgrim asgrim force-pushed the initial-boilerplate branch from 1342044 to c40af18 Compare February 29, 2024 08:33
composer.json Show resolved Hide resolved
@asgrim asgrim merged commit 7b23795 into main Mar 18, 2024
21 checks passed
@asgrim asgrim added this to the 0.1.0 - initial release milestone Aug 26, 2024
@asgrim asgrim deleted the initial-boilerplate branch October 3, 2024 19:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants