Skip to content

add clear comments and improve readability in HTMLProofer script#3555

Merged
jmeridth merged 1 commit intogithub:mainfrom
Maruti1887:patch-1
Dec 4, 2025
Merged

add clear comments and improve readability in HTMLProofer script#3555
jmeridth merged 1 commit intogithub:mainfrom
Maruti1887:patch-1

Conversation

@Maruti1887
Copy link
Contributor

@Maruti1887 Maruti1887 commented Nov 27, 2025

This commit adds detailed documentation comments and improves the formatting of the HTMLProofer runner script to enhance readability and make the code easier for new contributors to understand.

Key improvements:

  • Added explanatory comments for script purpose and each configuration option
  • Added section headers to structure the file
  • Documented URL ignore list and regex patterns
  • Improved spacing and formatting for clarity
  • Added frozen_string_literal: true for Ruby best practices

No functional behavior was changed; the script runs exactly as before. These updates strictly improve maintainability and contributor experience.

This commit adds detailed documentation comments and improves the formatting
of the HTMLProofer runner script to enhance readability and make the code
easier for new contributors to understand.

Key improvements:
- Added explanatory comments for script purpose and each configuration option
- Added section headers to structure the file
- Documented URL ignore list and regex patterns
- Improved spacing and formatting for clarity
- Added `frozen_string_literal: true` for Ruby best practices

No functional behavior was changed; the script runs exactly as before.
These updates strictly improve maintainability and contributor experience.
@Maruti1887 Maruti1887 requested a review from a team as a code owner November 27, 2025 15:45
@jmeridth jmeridth added this pull request to the merge queue Dec 4, 2025
Merged via the queue into github:main with commit 211617b Dec 4, 2025
3 checks passed
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.

3 participants