Comments (4)
staabm
commented on May 28, 2024
Maybe thats something you CS tool of choice should fix after runnung rector?
Edit: hmm thinking again about it.. it might be hard to detect this situation after the fact
from rector.
JoolsMcFly
commented on May 28, 2024
Hi Markus.
Yeah, it's going to be tough after refactoring to know where to insert _ to split words.
I feel that upper snake case (NEEDS_REVIEW) is more readable than joined upper case (NEEDSREVIEW) so it could be the default behaviour. On the other hand if we introduced a switch then we'd give users more control on the output.
What do you guys think?
from rector.
Related Issues (20)
- ReturnNeverTypeRector results in incompatible function signatures
- Skipper with match: */Expected* should not skip file named MatchParamTypeExpectedNameResolver.php HOT 3
- ReturnTypeFromStrictTypedCallRector infers static return type as static in caller
- fix invalid union return phpdocs with `void`. HOT 4
- can't set a new parameter to have no default value HOT 3
- PHP Fatal error: Uncaught Error: Typed static property RectorPrefix202312 HOT 2
- Documentation on rules is missing details on how to configure them HOT 3
- Incorrect behavior of SensitiveConstantNameRector, NullToStrictStringFuncCallArgRector HOT 1
- System error: "Scope not available on "PhpParser\Node\Expr\Array_" node, but is required by a refactorWithScope() method of "Rector\CodeQuality\Rector\Array_\CallableThisArrayToAnonymousFunctionRector HOT 5
- Rector does not refactor PHP8.3 ldap_connect deprecated signature
- Wrong variable name replacement for `$this` HOT 1
- Rector fails to convert dataProvider annotation to #[DataProvider] attribute for PHPUnit 10 HOT 3
- The filename, directory name, or volume label syntax is incorrect. HOT 10
- rector debug file HOT 1
- System error: Scope not available on PhpParser\Node\Expr\FuncCall node, but is required by a refactorWithScope() method HOT 1
- Incorrect behavior of TernaryFalseExpressionToIfRector
- [DX] Add RequiredPolyfillInterface to some rule to enable then, once the project is using polyfill package HOT 2
- Incorrect behavior of DowngradeMatchToSwitchRector HOT 2
- Error in SimplifyIfElseWithSameContentRector HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from rector.