Skip to content

Comments

languages/php: add formatting#1401

Merged
horriblename merged 2 commits intoNotAShelf:mainfrom
snoweuph:feature/php-cs
Feb 11, 2026
Merged

languages/php: add formatting#1401
horriblename merged 2 commits intoNotAShelf:mainfrom
snoweuph:feature/php-cs

Conversation

@snoweuph
Copy link
Contributor

@snoweuph snoweuph commented Feb 7, 2026

php formatting

Sanity Checking

  • I have updated the changelog as per my changes
  • I have tested, and self-reviewed my code
  • My changes fit guidelines found in hacking nvf
  • Style and consistency
    • I ran Alejandra to format my code (nix fmt)
    • My code conforms to the editorconfig configuration of the project
    • My changes are consistent with the rest of the codebase
  • If new changes are particularly complex:
    • My code includes comments in particularly complex areas
    • I have added a section in the manual
    • (For breaking changes) I have included a migration guide
  • Package(s) built:
    • .#nix (default package)
    • .#maximal
    • .#docs-html (manual, must build)
    • .#docs-linkcheck (optional, please build if adding links)
  • Tested on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin

Add a 👍 reaction to pull requests you find important.

github-actions bot pushed a commit that referenced this pull request Feb 8, 2026
@github-actions
Copy link

github-actions bot commented Feb 8, 2026

🚀 Live preview deployed from 4d0d188

View it here:

Debug Information

Triggered by: horriblename

HEAD at: feature/php-cs

Reruns: 2090

@horriblename horriblename enabled auto-merge (squash) February 11, 2026 12:01
github-actions bot pushed a commit that referenced this pull request Feb 11, 2026
@horriblename horriblename merged commit bd265fe into NotAShelf:main Feb 11, 2026
14 checks passed
@snoweuph snoweuph deleted the feature/php-cs branch February 15, 2026 14:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants