Skip to content

CI and dev environment configuration #666

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

CI and dev environment configuration #666

wants to merge 1 commit into from

Conversation

GKFX
Copy link
Contributor

@GKFX GKFX commented Oct 6, 2020 •
edited
Loading

Subset of #663. This pull request:

  • Makes the CI not dependant on a PHPStan pass so that it can run through to the unit tests
  • Adds alcaeus/mongo-php-adapter so that composer install can run without ext-mongo. This could be removed again if doctrine/mongodb-odm is upgraded, as in the elchris fork.
  • Disables the Propel tests as all the Propel code seems to be missing something in the FOS\OAuthServerBundle\Propel\om namespace
  • Updates the PHPStan configuration including the addition of phpstan-symfony.

GKFX added a commit to GKFX/FOSOAuthServerBundle that referenced this pull request Oct 8, 2020
@GKFX
Remove most warnings and deprecations from tests
efbcd33 
This commit is dependant on FriendsOfSymfony#663 or FriendsOfSymfony#666 to upgrade the PHPUnit
dependency to ^8. It brings the tests to a passing state by removing all
deprecation warnings that are within this project's control.
@GKFX GKFX mentioned this pull request Oct 8, 2020
Open
GKFX added a commit to GKFX/FOSOAuthServerBundle that referenced this pull request Oct 8, 2020
@GKFX
Remove most warnings and deprecations from tests
76f2f54 
This commit is dependant on FriendsOfSymfony#663 or FriendsOfSymfony#666 to upgrade the PHPUnit
dependency to ^8. It brings the tests to a passing state by removing all
deprecation warnings that are within this project's control.
# for free to join this conversation on GitHub. Already have an account? # 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.
1 participant
@GKFX