Skip to content
This repository was archived by the owner on Jul 23, 2025. It is now read-only.

Add linting and format checks#20

Merged
danbarr merged 8 commits intomainfrom
add-linting
Jan 7, 2025
Merged

Add linting and format checks#20
danbarr merged 8 commits intomainfrom
add-linting

Conversation

@danbarr
Copy link
Collaborator

@danbarr danbarr commented Jan 7, 2025

Adds configs for Prettier, markdownlint, and ESLint. Also adds GitHub Actions to run them and a test build on PRs.

@vercel
Copy link

vercel bot commented Jan 7, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
codegate-docs 🛑 Canceled (Inspect) Jan 7, 2025 3:50pm

@danbarr
Copy link
Collaborator Author

danbarr commented Jan 7, 2025

@peppescg I cribbed some of this from your additions to codegate-ui, so tagging you for a review 😄

@peppescg
Copy link

peppescg commented Jan 7, 2025

That's amazing @danbarr 🚀

Copy link

@peppescg peppescg left a comment

Choose a reason for hiding this comment

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

I have question, usually in FE repo we use husky for adding precommit hook, checking eslint, typescript issues before commit...I don't know if it could be helpful also here...in case you consider it helpful I can help on this ( consider that in codegate-ui I installed it last week )

peppescg
peppescg previously approved these changes Jan 7, 2025
Copy link

@peppescg peppescg left a comment

Choose a reason for hiding this comment

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

Great work 👏
I would add a precommit hook and a script for markdown lint, having the tools during the development flow too, but it could be added in a next PR too, so I approve 👍

JAORMX
JAORMX previously approved these changes Jan 7, 2025
@danbarr danbarr dismissed stale reviews from JAORMX and peppescg via a573720 January 7, 2025 15:30
Copy link

@peppescg peppescg left a comment

Choose a reason for hiding this comment

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

👏 great job

@danbarr danbarr merged commit 6d4f322 into main Jan 7, 2025
4 checks passed
@danbarr danbarr deleted the add-linting branch January 7, 2025 16:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants