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

Method chaining must be fully multiline, with the first item on a new line #7793

Open
Tracked by #7247
mvorisek opened this issue Jan 31, 2024 · 4 comments
Open
Tracked by #7247
Labels
kind/feature request status/to verify issue needs to be confirmed or analysed to continue

Comments

@mvorisek
Copy link
Contributor

mvorisek commented Jan 31, 2024

Feature request

extracted from #7247

expected output:

 <?php

-$foo->a()->b()->c();
+$foo
+    ->a()
+    ->b()
+    ->c();

helpful also for coverage measurements

@Wirone
Copy link
Member

Wirone commented Jan 31, 2024

I've linked this issue in the todo list of #7247 but please don't extract other items to dedicated issues. PER-CS support doesn't need to be provided at once (rather PR per todo list item), but also atomic PRs don't need atomic issues, and can be referenced to the main one. Some todo items have issues/PRs linked because these were already created. Creating new ones will only cause more maintenance burden (stale bot) and will increase the size of the backlog.

This comment has been minimized.

This comment has been minimized.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale May 30, 2024
@mvorisek

This comment has been minimized.

@Wirone Wirone reopened this May 30, 2024
@github-actions github-actions bot added status/to verify issue needs to be confirmed or analysed to continue and removed status/stale labels May 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature request status/to verify issue needs to be confirmed or analysed to continue
Projects
None yet
Development

No branches or pull requests

2 participants