Skip to content

Conform Pony source to style guide#73

Merged
SeanTAllen merged 1 commit intomainfrom
sean/conform-to-style-guide
Mar 1, 2026
Merged

Conform Pony source to style guide#73
SeanTAllen merged 1 commit intomainfrom
sean/conform-to-style-guide

Conversation

@SeanTAllen
Copy link
Copy Markdown
Member

Fix all pony-lint violations across all Pony source files: line-length,
blank-lines, assignment-indent, array-literal-format, comment-spacing,
dot-spacing, indentation, lambda-spacing, file-naming, package-docstring,
call-argument-format, public-docstring, and control-structure-alignment.

File renames per the file-naming rule:

  • windows_codepages.pony -> windows_code_pages.pony
  • simple-example.pony -> main.pony

Added a pony-lint CI workflow (.github/workflows/pony-lint.yml) that
triggers on PRs touching *.pony files to enforce the style guide going
forward.

@ponylang-main ponylang-main added the discuss during sync Should be discussed during an upcoming sync label Feb 28, 2026
Fix all pony-lint violations across all Pony source files:
line-length, blank-lines, assignment-indent, array-literal-format,
comment-spacing, dot-spacing, indentation, lambda-spacing,
file-naming, package-docstring, call-argument-format,
public-docstring, and control-structure-alignment.

File renames per the file-naming rule:
- windows_codepages.pony -> windows_code_pages.pony
- simple-example.pony -> main.pony

Add a pony-lint CI workflow that triggers on PRs touching
*.pony files to enforce the style guide going forward.
@SeanTAllen SeanTAllen force-pushed the sean/conform-to-style-guide branch from c98e636 to 848bc66 Compare February 28, 2026 23:33
@SeanTAllen SeanTAllen merged commit a3c436b into main Mar 1, 2026
11 checks passed
@SeanTAllen SeanTAllen deleted the sean/conform-to-style-guide branch March 1, 2026 00:11
@ponylang-main ponylang-main removed the discuss during sync Should be discussed during an upcoming sync label Mar 1, 2026
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