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

Fix multiline if statement indentation #71

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Gleek
Copy link
Contributor

@Gleek Gleek commented Jun 20, 2024

Currently a multiline if statement is indented as follows:

if ($testing &&
$testing2
&& true
) {
    // do something
}

But instead should be indented as:

if ($testing &&
    $testing2
    && true
) {
    // do something
}

This change fixes this.

Currently a multiline if statement is indented as follows:
```php
if ($testing &&
$testing2
&& true
) {
    // do something
}
```

But instead should be indented as:
```php
if ($testing &&
    $testing2
    && true
) {
    // do something
}
```

This change fixes this.
@piotrkwiecinski
Copy link
Contributor

@Gleek @zonuexe probably we should deprecate this repository. Emacs 30 has php-ts-mode recently merged https://debbugs.gnu.org/cgi/bugreport.cgi?bug=71380.
You can see it in mirror https://github.com/emacs-mirror/emacs/blob/master/lisp/progmodes/php-ts-mode.el

Any extra contributions should be done directly as patches to emacs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants