Skip to content

CI Github Action Improvement - #84

Merged
luciankt merged 5 commits into
csvistool:masterfrom
karkir0003:auto-linter-github-action-improvement
Oct 1, 2026
Merged

luciankt merged 5 commits into
csvistool:masterfrom
karkir0003:auto-linter-github-action-improvement

Conversation

@karkir0003

@karkir0003 karkir0003 commented Sep 27, 2026 •

Copy link
Copy Markdown
Contributor

Context

In #83 , I attempted to create a GitHub action to automate the npm run lint and npm run prettier checks. However, after #83 got merged, I realized that there is a better setup that can simplify the problem. This PR tries to make things right

Changes Made

  • Updated the ci.yaml to act as a validation layer for enforcing the formatting standards as outlined in the contribution guidelines
  • Updated CONTRIBUTING.md to bring awareness to the automated hook to auto apply prettier and linting formatting standards
  • Created a .husky pre-commit hook that on commit automatically lints the staged files! No need to remember to manually run npm run lint and npm run prettier

Test Plan

  • Dry run of the CI Action here
    1. Created a new branch off of this one with the improved action
    2. Ran npm run lint && npm run prettier
    3. git push
    4. Create PR as seen above and the CI run succeeds

@karkir0003

Copy link
Copy Markdown
Contributor Author

@luciankt , can you review this! I made an update to the CI improvement process to better streamline the process. Let me know your thoughts

@karkir0003 karkir0003 changed the title Github Action for CI Improvement CI Github Action Improvement Sep 27, 2026
@luciankt
luciankt merged commit 32a2b46 into csvistool:master Oct 1, 2026
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.

2 participants