Skip to content

Add linter from HS2-Software#6

Open
VerisimilitudeX wants to merge 5 commits intomainfrom
linter
Open

Add linter from HS2-Software#6
VerisimilitudeX wants to merge 5 commits intomainfrom
linter

Conversation

@VerisimilitudeX
Copy link
Member

No description provided.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds a GitHub Actions CI workflow that runs cpplint, a C++ style linter, on the repository's source code. It is intended to enforce code style standards by scanning C++ files on pushes and pull requests targeting main and develop branches.

Changes:

  • Adds a new cpplint.yml GitHub Actions workflow that installs and runs cpplint with --recursive and various --exclude options.
  • Configures concurrency settings to cancel in-progress runs when a newer run is triggered on the same PR.
  • Sets the workflow to run on push/PR events targeting main and develop, with paths-ignore for docs and markdown files.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@MahirEmran
Copy link
Contributor

Error here in actions was fixed with PR #5 , so I think if you pull from main that should fix it. Though unsure if my code passes linter haha.

@VerisimilitudeX
Copy link
Member Author

Thanks @MahirEmran, pulling from main fixed it. @j4lando should be ready to review whenever you’re free (I know you’ve got midterms going on so no rush, gl!)

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.

4 participants