Skip to content

Conversation

@junichi11
Copy link
Member

@junichi11 junichi11 added the PHP [ci] enable extra PHP tests (php/php.editor) label Jul 9, 2020
@junichi11 junichi11 added this to the 12.1 milestone Jul 9, 2020
@junichi11 junichi11 requested a review from tmysik July 9, 2020 05:22
@tmysik
Copy link
Member

tmysik commented Jul 9, 2020

@junichi11

Just thinking whether it would make sense to make it configurable... 🤔 Probably not necessary :)

BTW can I merge it, right? The test failure seems unrelated to me. Or am I wrong?

Thanks!

@junichi11
Copy link
Member Author

@tmysik

Just thinking whether it would make sense to make it configurable... Probably not necessary :)

Let's think that after the new level is added in the future again :) e.g. We can add the max level combobox. or change to the text field.

BTW can I merge it, right? The test failure seems unrelated to me. Or am I wrong?

It should be OK. Maybe this: https://lists.apache.org/thread.html/r680b5a50fc7c8d07c91c0750df6cafa1e49732cbfdfbc2aa15c3c012%40%3Cdev.netbeans.apache.org%3E

Thank you!

@tmysik
Copy link
Member

tmysik commented Jul 9, 2020

@junichi11

We can add the max level combobox. or change to the text field.

I was thinking more about a simple configuration (properties) file with a max level. Anyone could update it manually without a code change. And yes, if someone puts there 1000 then it is his/her fault :)

@tmysik tmysik merged commit 86e29e5 into apache:master Jul 9, 2020
@junichi11
Copy link
Member Author

@tmysik

I was thinking more about a simple configuration (properties) file with a max level. Anyone could update it manually without a code change. And yes, if someone puts there 1000 then it is his/her fault :)

Makes sense. Good idea!

@junichi11 junichi11 deleted the netbeans-4652-phpstan-add-level8 branch July 9, 2020 13:23
@junichi11 junichi11 changed the title [NETBEANS-4652] Change the max level of PHPStan to 8 [NETBEANS-4562] Change the max level of PHPStan to 8 Jul 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PHP [ci] enable extra PHP tests (php/php.editor)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants