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

fix for php 7.4 builds in github workflow #828

Closed
wants to merge 1 commit into from

Conversation

elizoller
Copy link
Member

What does this Pull Request do?

Fixes the github workflow for running tests. At least for PHP 7.4

What's new?

  • specifying the minimum stability for composer packages

How should this be tested?

  • if the tests run and pass then this is good.

Interested parties

@Islandora/8-x-committers @mjordan

@whikloj
Copy link
Member

whikloj commented Mar 11, 2021

@elizoller can you explain why this is necessary? Just asking for my own knowledge.

@elizoller
Copy link
Member Author

the default is stable i believe. and certain modules don't have stable releases. ie. context which only has a beta release (https://www.drupal.org/project/context)

@elizoller
Copy link
Member Author

@whikloj or @mjordan any interest in review? tests are green

@mjordan
Copy link

mjordan commented Mar 11, 2021

I can, but I'm now in calls all the way through to 2 PM Pacific. Can try to do it then or this evening.

@elizoller elizoller mentioned this pull request Mar 12, 2021
@elizoller
Copy link
Member Author

superseded by d6ceebd

@elizoller elizoller closed this Mar 12, 2021
@mjordan
Copy link

mjordan commented Mar 14, 2021

FYI I merged recent updates to 8.x-1.x into my PR branch and he build for PHP 7.4 drupal 9.1.5 is now passing.

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.

3 participants