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

Prepare for os2forms 3.16 #12

Merged
merged 7 commits into from
Sep 26, 2024
Merged

Conversation

jekuaitk
Copy link
Collaborator

@jekuaitk jekuaitk commented Sep 24, 2024

Link to ticket

https://leantime.itkdev.dk/tickets/showKanban?search=true&projectId=41&users=21&sprint=all#/tickets/showTicket/2247

Description

Upgrades to Drupal 10 via os2form/os2forms:^3.16 alongside an upgrade to PHP 8.3

Coding standards

The Drupal coder standard is now case sensitive imports.

@@ -23,7 +23,7 @@ jobs:
runs-on: ubuntu-latest
strategy:
matrix:
php-versions: [ '8.1' ]
php-versions: [ '8.2' ]
Copy link
Contributor

Choose a reason for hiding this comment

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

Why not 8.3?

README.md Outdated
@@ -68,8 +68,8 @@ Check coding standards:

```sh
// PHP CS Fixer
docker run --rm --interactive --tty --volume ${PWD}:/app itkdev/php8.1-fpm:latest composer install
docker run --rm --interactive --tty --volume ${PWD}:/app itkdev/php8.1-fpm:latest composer coding-standards-check
docker run --rm --interactive --tty --volume ${PWD}:/app itkdev/php8.2-fpm:latest composer install
Copy link
Contributor

Choose a reason for hiding this comment

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

Why not 8.3?

@jekuaitk jekuaitk merged commit 6c0fdd5 into develop Sep 26, 2024
8 checks passed
@jekuaitk jekuaitk deleted the feature/upgrade-os2forms-requirement branch September 26, 2024 12:13
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