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

Update .scrutinizer.yml #65

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

KarlComSe
Copy link

Lyckas inte köra analys-noden utan att specificera PHP för den. Scrutinizer går tillbaka till PHP7 och composer 1.

Lyckas inte köra analys-noden utan att specificera PHP för den. Scrutinizer går tillbaka till PHP7 och composer 1.
@mosbth
Copy link
Member

mosbth commented May 17, 2024

Jag lyckas köra allt med den konfigfil som ligger i kursrepot.
https://github.com/dbwebb-se/mvc/blob/main/example/scrutinizer/.scrutinizer.yml#L14

"Funkar för mig".
https://scrutinizer-ci.com/g/mosbth/mvc-2024-scrutinizer/inspections

Min konfigfil, som är lik den som ligger i kursrepot.
https://github.com/mosbth/mvc-2024-scrutinizer/blob/main/.scrutinizer.yml

Eller?

@KarlComSe
Copy link
Author

Ja, det verkar fungera för dig. Av för mig ännu ej känd anledning så körs din analys med PHP 8.3.3, trots att det inte är specificerat i konfig-filen för analysis-noden.

Mitt resultat (test setup failed):
https://scrutinizer-ci.com/g/KarlComSe/mvc-report/inspections/9a4ebaaf-f176-4cf9-92ea-da206c8c0be9

Här kan man se att det inte gick, och de olika konfigurationsfilerna:
https://github.com/KarlComSe/mvc-report/commits/main/.scrutinizer.yml

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