-
-
Notifications
You must be signed in to change notification settings - Fork 352
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
ci: pre-commit & generate actions #162
base: master
Are you sure you want to change the base?
Conversation
mdeweerd
commented
Jan 5, 2024
•
edited
Loading
edited
- Add pre-commit (performing linting & formatting of certain files (not committed, on ly reported)).
- Add action to generate documentation and autotools on updates.
57118ae
to
9ed2201
Compare
9ed2201
to
e0a4001
Compare
Hi @ashamedbit, would you take a look at this please? I'm the DM of the shc Debian package, and shc is not compiling under it (see #167). |
e0a4001
to
c2c263a
Compare
No, #161 does not superseed this. More often than not, project owners want a seperate commit for everything. So this is the commit for the flow, the other is a commit for fixes. I just now merge and pushd the ci flow to check the other branch and removed it again. Out of curiosity, I also checked #165 with the flow and some memory issues are still reporte by the linting tool: |
Yeah, that's a major issue, I'll give @ashamedbit a few days to see how he'd response... |
5662a5f
to
c2c263a
Compare