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

chore(cicd): adds workflow to run checks on push to main and pull-reqs #537

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

doublethefish
Copy link
Contributor

This configures the submodule and runs npm run check on the code.

This is a first-pass of getting the checks working before I make any other changes, and help me ensure I don't break the project.

doublethefish and others added 4 commits November 12, 2024 06:54
…pe-assertion

... as the extra asserts in this code base seem more useful

My personal preference is to be explicit with typing and this eslint
check is counter to that. Also in the areas where this linting error are
triggered it seems useful to have the type asserts as they help
code-clarity
… push and PRs

This should help ensure the code-quality stays as good as it already is, and/or improve it over time.
This helps fix a gotcha for future contributors.
This will get the workflow running without code-infra errors, but it
will not pass linting, yet.
@doublethefish doublethefish changed the title chore(cicd): add workflow to run checks on push to main and pull-reqs chore(cicd): adds workflow to run checks on push to main and pull-reqs Nov 12, 2024
@doublethefish
Copy link
Contributor Author

For the workflow checks to pass we will require vrtmrz/livesync-commonlib#7 to be accepted upstream, and the git-submodule updated with that change.

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.

1 participant