Skip to content

Enhance CI workflow with test execution and job updates#923

Open
Shane32 wants to merge 7 commits intomdx-editor:mainfrom
Shane32:add-test-action
Open

Enhance CI workflow with test execution and job updates#923
Shane32 wants to merge 7 commits intomdx-editor:mainfrom
Shane32:add-test-action

Conversation

@Shane32
Copy link
Copy Markdown
Contributor

@Shane32 Shane32 commented Apr 13, 2026

Summary

Adds a GitHub Actions CI workflow that runs on every pull request, with three parallel jobs:

  • Test: runs lint, typecheck, and npm run test:once
  • Build JS: runs npm run build
  • Build Docs: runs npm run build:docs:api

Comment thread vite.config.ts Outdated
@Shane32
Copy link
Copy Markdown
Contributor Author

Shane32 commented Apr 19, 2026

Adding this workflow makes it easier to see if a PR will match linting requirements by the repo, and confirm it builds properly.

@Shane32
Copy link
Copy Markdown
Contributor Author

Shane32 commented Apr 19, 2026

I could add prettier checks also, if desired. I usually use prettier . --check for my repos.

@Shane32
Copy link
Copy Markdown
Contributor Author

Shane32 commented Apr 20, 2026

I could add prettier checks also, if desired. I usually use prettier . --check for my repos.

It seems that prettier is enforced via an eslint plugin so a separate prettier check would be redundant.

@petyosi
Copy link
Copy Markdown
Contributor

petyosi commented Apr 23, 2026

Thank you, sorry for the delay on my response, this is necessary indeed. I will look into this.

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.

3 participants