Skip to content

Conform Pony source to style guide - #73

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

SeanTAllen merged 1 commit into
mainfrom
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