Reduce required PHP version to 7.4 again #4

Open
akirk wants to merge 2 commits from akirk/patch-1 into main
akirk commented 2023-06-18 19:20:19 +00:00 (Migrated from github.com)

In 2.1.0 the required PHP version was increased to 8.0, was there a particular reason for that except that PHP 7.4 is EOL? It still parses fine. If no, can we reduce it again?

❯ php -v
PHP 7.4.33 (cli) (built: Jun 17 2023 06:19:58) ( NTS )
Copyright (c) The PHP Group
Zend Engine v3.4.0, Copyright (c) Zend Technologies
with Zend OPcache v7.4.33, Copyright (c), by Zend Technologies
with Xdebug v3.1.3, Copyright (c) 2002-2022, by Derick Rethans
❯ php -l biscotti.php
No syntax errors detected in biscotti.php

In [2.1.0](https://github.com/boogah/biscotti/releases/tag/v2.1.0) the required PHP version was increased to 8.0, was there a particular reason for that except that PHP 7.4 is EOL? It still parses fine. If no, can we reduce it again? ❯ php -v PHP 7.4.33 (cli) (built: Jun 17 2023 06:19:58) ( NTS ) Copyright (c) The PHP Group Zend Engine v3.4.0, Copyright (c) Zend Technologies with Zend OPcache v7.4.33, Copyright (c), by Zend Technologies with Xdebug v3.1.3, Copyright (c) 2002-2022, by Derick Rethans ❯ php -l biscotti.php No syntax errors detected in biscotti.php
This pull request can be merged automatically.
You are not authorized to merge this pull request.

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin akirk/patch-1:akirk/patch-1
git checkout akirk/patch-1
Sign in to join this conversation.
No Reviewers
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: boogah/biscotti#4
No description provided.