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.

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

Fix pipeline issues #639

Merged
merged 23 commits into from
Dec 27, 2021
Merged

Fix pipeline issues #639

merged 23 commits into from
Dec 27, 2021

Conversation

Sebbo94BY
Copy link
Collaborator

@Sebbo94BY Sebbo94BY commented Dec 4, 2021 •
edited
Loading

This should fix various pipeline issues.

composer-require-check: unknown symbol

> composer-require-checker check ./composer.json
ComposerRequireChecker 3.6.0@e73c0bb75522d861aef3e8d58991d50bf0125a0b
The following 1 unknown symbols were found:
+-----------------+--------------------+
| Unknown Symbol  | Guessed Dependency |
+-----------------+--------------------+
| IMAP\Connection |                    |
+-----------------+--------------------+
Script composer-require-checker check ./composer.json handling the static-analysis event returned with error code 1

This symbol is only known in PHP 8.1 and newer.

Fix various php-cs-fixer issues.

Note: PHP 8.1 is not fully supported yet by PHP-CS-Fixer: PHP-CS-Fixer/PHP-CS-Fixer#5891

Fix coverage issue by defining the secret again in the workflow config. See #473 for further information.

Reduce test jobs by only running on a specific Ubuntu version instead of specific plus latest one.

@Sebbo94BY Sebbo94BY marked this pull request as draft December 4, 2021 02:06
Sebbo94BY pushed a commit that referenced this pull request Dec 4, 2021
Remove composer require checker to fix pipeline
0677593 
Relates to PR #639 and can not be fixed at the moment as it's a false-positive.
@Sebbo94BY Sebbo94BY mentioned this pull request Dec 4, 2021
Merged
@Sebbo94BY Sebbo94BY force-pushed the Fix-pipeline-issues branch 3 times, most recently from a27d7c9 to 60e33c0 Compare December 4, 2021 16:11
@Sebbo94BY Sebbo94BY changed the title Fix unknown symbols in pipeline Fix pipeline issues Dec 4, 2021
@Sebbo94BY Sebbo94BY force-pushed the Fix-pipeline-issues branch 5 times, most recently from deb2a6f to 7276ea1 Compare December 4, 2021 19:07
@Sebbo94BY Sebbo94BY mentioned this pull request Dec 4, 2021
Merged
Sebbo94BY pushed a commit that referenced this pull request Dec 14, 2021
Remove composer require checker to fix pipeline
35b8b35 
Relates to PR #639 and can not be fixed at the moment as it's a false-positive.
Sebastian Krätzig added 6 commits December 14, 2021 01:07
Fix unknown symbols in pipeline
c292af8
Only test a specific Ubuntu version
4cb2f18 
Reduce amount of jobs and avoid possible failing jobs due to unknown changes in the OS.
473: Re-add secret to fix coverage issue
6324447 
This needs to be migrated to Github secrets by the project owner. Due to no feedback after over one year, we had to revert it to fix the pipeline error.
Add some newlines to workflow config for better readability
f86af59
Fix php-cs-fixer issues
5f67575
Remove dynamic PHP matrix due to dependency issues
1d02f53
@Sebbo94BY Sebbo94BY marked this pull request as ready for review December 27, 2021 23:21
@Sebbo94BY Sebbo94BY merged commit 28a7820 into develop Dec 27, 2021
@Sebbo94BY Sebbo94BY deleted the Fix-pipeline-issues branch December 27, 2021 23:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@Sebbo94BY