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.

Sign up for GitHub

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

Jump to bottom

Ensure that all code lines are shorter than 72 characters #478

Merged
merged 7 commits into from
Aug 13, 2024
Merged

Ensure that all code lines are shorter than 72 characters #478

merged 7 commits into from
Aug 13, 2024

Conversation

Witiko
Copy link
Owner

@Witiko Witiko commented Aug 12, 2024 •
edited
Loading

This PR makes the following changes:

  • Ensure that all code lines are shorter than 72 characters.
    This prevents overflows of the typeset code in the technical documentation.
  • Add pseudo-target check-line-length to Makefile.
  • Add checking of line length to continuous integration.
  • Update CHANGES.md.

This PR also makes the following changes:

  • Add pseudo-target check-tabs-and-spaces to Makefile.
  • Fix the definition of \markdownInput with frozenCache enabled.

@Witiko Witiko added technical debt Related to code smells and refactoring continuous integration Related to the continuous integration labels Aug 12, 2024
@Witiko Witiko added this to the 3.7.0 milestone Aug 12, 2024
@Witiko Witiko marked this pull request as ready for review August 12, 2024 21:44
@Witiko Witiko force-pushed the feat/enforce-line-length branch 2 times, most recently from 1e5825a to 565eb1b Compare August 12, 2024 21:47
@Witiko Witiko force-pushed the feat/enforce-line-length branch from 565eb1b to 9d49fb3 Compare August 12, 2024 22:41
Witiko
Witiko commented Aug 13, 2024
View reviewed changes
markdown.dtx Outdated Show resolved Hide resolved
markdown.dtx Outdated Show resolved Hide resolved
markdown.dtx Outdated Show resolved Hide resolved
markdown.dtx Outdated Show resolved Hide resolved
markdown.dtx Outdated Show resolved Hide resolved
markdown.dtx Outdated Show resolved Hide resolved
markdown.dtx Outdated Show resolved Hide resolved
markdown.dtx Outdated Show resolved Hide resolved
markdown.dtx Outdated Show resolved Hide resolved
markdown.dtx Outdated Show resolved Hide resolved
@Witiko Witiko self-assigned this Aug 13, 2024
@Witiko Witiko merged commit a326478 into main Aug 13, 2024
11 checks passed
@Witiko Witiko deleted the feat/enforce-line-length branch August 13, 2024 08:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@Witiko Witiko

Labels
continuous integration Related to the continuous integration technical debt Related to code smells and refactoring
Projects
None yet
Milestone
3.7.0
Development

Successfully merging this pull request may close these issues.
1 participant
@Witiko