Skip to content

Phpstan integration#720

Merged
laurit17 merged 18 commits into
mainfrom
lauri/php
Apr 2, 2024
Merged

Phpstan integration#720
laurit17 merged 18 commits into
mainfrom
lauri/php

Conversation

@laurit17
Copy link
Copy Markdown
Contributor

@laurit17 laurit17 commented Mar 28, 2024

  • php config definitions

@trunk-io
Copy link
Copy Markdown

trunk-io Bot commented Mar 28, 2024

⏱️ 2h 5m total CI duration on this PR
Job Cumulative Duration Recent Runs
Linter Tests macOS 54m 🟥🟩🟩🟩🟩 (+1 more)
Linter Tests Linux 24m 🟩🟥🟥🟥🟩 (+3 more)
Tool Tests (macOS) 11m 🟩🟩🟩🟩🟩 (+1 more)
CodeQL-Build 8m 🟩🟩🟩🟩🟩 (+1 more)
Tool Tests (Linux) 7m 🟩🟩🟩🟩🟩 (+1 more)
Trunk Check runner [linux] 6m 🟩🟩🟩🟩🟩 (+1 more)
Windows Linter Tests 6m 🟩🟩🟩🟩🟩 (+1 more)
Action Tests 5m 🟩🟩🟩🟩🟩 (+1 more)
Repo Tests / Plugin Tests 3m 🟩🟩🟩🟩🟩 (+1 more)
Detect changed files 47s 🟩🟩🟩🟩🟩 (+1 more)
Aggregate Test Results 23s 🟩🟥🟥🟥🟩 (+3 more)

settingsfeedbackdocs ⋅ learn more about trunk.io

@laurit17 laurit17 requested review from TylerJang27 and det March 28, 2024 22:25
Copy link
Copy Markdown
Collaborator

@TylerJang27 TylerJang27 left a comment

Choose a reason for hiding this comment

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

Some cleanup and sanity check comments

Comment thread .github/actions/linter_tests/action.yaml
Comment thread linters/phpstan/plugin.yaml Outdated
Comment thread linters/phpstan/phpstan.test.ts Outdated
Comment thread runtimes/php/plugin.yaml
Comment on lines +6 to +8
downloads:
- version: 2.5.1
gpg: CBB3D576F2A0946F
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Is this missing a URL or is this sufficient?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

this is sufficient - we are just using the gpg key to allow another php runtime tool to download this

@laurit17 laurit17 requested a review from TylerJang27 April 1, 2024 23:40
Copy link
Copy Markdown
Collaborator

@TylerJang27 TylerJang27 left a comment

Choose a reason for hiding this comment

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

LGTM

@laurit17 laurit17 merged commit e1744ad into main Apr 2, 2024
@laurit17 laurit17 deleted the lauri/php branch April 2, 2024 17:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants