laravel-enum icon indicating copy to clipboard operation
laravel-enum copied to clipboard

Add constant visibility for PSR-12 compatiblity

Open hailwood opened this issue 5 years ago • 3 comments

Description

Makes the enum:make command add the public visibility keyword to all generated constants.

Motivation and context

https://www.php-fig.org/psr/psr-12/#43-properties-and-constants

How has this been tested?

Updated test cases

Types of changes

What types of changes does your code introduce? Put an x in all the boxes that apply:

  • [ ] Bug fix (non-breaking change which fixes an issue)
  • [x] New feature (non-breaking change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

Go over all the following points, and put an x in all the boxes that apply.

Please, please, please, don't send your pull request until all of the boxes are ticked. Once your pull request is created, it will trigger a build on our continuous integration server to make sure your tests and code style pass.

  • [x] I have read the CONTRIBUTING document.
  • [x] My pull request addresses exactly one patch/feature.
  • [x] I have created a branch for this patch/feature.
  • [ ] Each individual commit in the pull request is meaningful.
  • [x] I have added tests to cover my changes.
  • [x] If my change requires a change to the documentation, I have updated it accordingly.

If you're unsure about any of these, don't hesitate to ask. We're here to help!

hailwood avatar Sep 15 '19 22:09 hailwood

I did this quickly online, so of course the commits are a bit dirty. Good thing Github has the Squash merge option ;)

hailwood avatar Sep 15 '19 22:09 hailwood

If we will provide PSR-12 compatibility, it should require all PSR-12 rules. For now, we haven't automated tools to check it.

slavarazum avatar Sep 21 '19 11:09 slavarazum

That's probably a fair point. I wasn't actually meaning PSR-12 compat for the library, just for the generated enums. This was the only thing PhpStorm flagged on the generated files after enabling the PSR-12 ruleset.

I guess we could try running styleci or PHPCS over the fixtures for the tests to verify PSR-12 compatibility.

hailwood avatar Sep 21 '19 11:09 hailwood