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.

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

Add ESLint Auto Fix for Fixable Issues in CI #791

Merged
merged 2 commits into from
Jul 13, 2023

Conversation

pjkaufman
Copy link
Collaborator

Fixes #782

Changes Made:

  • Added logic for auto fixing fixable ESLint errors via the CI which should cut down on PRs requiring formatting changes

pjkaufman and others added 2 commits July 12, 2023 20:34
* first attempt at autocommit to fix eslint errors

* eslint error added

* updated main workflow

* simplify tests action and try with checking things out

* skip checkout to see if that fixes CI issues

* remove matrix strategy reference to see if that is the problem

* removed forced branch name that I overlooked

* undid some changes made in order to make it more like it was prior to see if it works

* Bot: fix ESLint errors

---------

Co-authored-by: pjkaufman <[email protected]>
@pjkaufman pjkaufman self-assigned this Jul 13, 2023
@pjkaufman pjkaufman merged commit bc8535d into platers:master Jul 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

FR: Use Eslint in CI to directly fix and push style issues into repo
1 participant