Skip to content

ATS-984 Use pre-commit to fix and prevent whitespace issues - #540

Open
mteodori wants to merge 15 commits into
masterfrom
ATS-pre-commit
Open

ATS-984 Use pre-commit to fix and prevent whitespace issues#540
mteodori wants to merge 15 commits into
masterfrom
ATS-pre-commit

Conversation

@mteodori

@mteodori mteodori commented Feb 23, 2022

Copy link
Copy Markdown
Member
  • Add pre-commit to check json syntax for travis, github actions and dependabot
  • fix mixed-line-ending
  • exclude gimp files
  • end-of-file-fixer
  • add trailing-whitespace
  • fix trailing-whitespace
  • Add check-yaml
  • fix trailing-whitespace
  • add editorconfig and checker

Ref. ATS-984

@pr-triage pr-triage Bot added the PR: draft label Feb 23, 2022
@mteodori mteodori changed the title ATS pre commit Use pre-commit to fix and prevent syntax issues Feb 24, 2022
@mteodori
mteodori requested a review from alandavis February 24, 2022 10:54
@mteodori
mteodori marked this pull request as ready for review February 24, 2022 11:14
@mteodori mteodori changed the title Use pre-commit to fix and prevent syntax issues ATS-984 Use pre-commit to fix and prevent whitespace issues Feb 24, 2022

@tpage-alfresco tpage-alfresco left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These changes look good to me.

@killerboot
killerboot removed their request for review September 4, 2023 16:04
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