Skip to content

Comments

FEAT: add prek,schema validation and GHA#25

Merged
maxulysse merged 9 commits intoaskimed:mainfrom
maxulysse:prek
Feb 13, 2026
Merged

FEAT: add prek,schema validation and GHA#25
maxulysse merged 9 commits intoaskimed:mainfrom
maxulysse:prek

Conversation

@maxulysse
Copy link
Collaborator

@maxulysse maxulysse commented Feb 13, 2026

so that #24 is never needed ever again

Adds:

  • prek
  • GHA for linting
  • schema and schema validation in GHA

Copy link
Collaborator

@nvnieuwk nvnieuwk left a comment

Choose a reason for hiding this comment

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

Might also be a good idea to add a JSON schema validator in the future

Co-authored-by: Matthias Hörtenhuber <mashehu@users.noreply.github.com>
@mashehu
Copy link
Collaborator

mashehu commented Feb 13, 2026

how about also a json-schema for that specific format?

@maxulysse
Copy link
Collaborator Author

how about also a json-schema for that specific format?

I was already on it, @nvnieuwk asked me to

Co-authored-by: Matthias Hörtenhuber <mashehu@users.noreply.github.com>
Co-authored-by: Matthias Hörtenhuber <mashehu@users.noreply.github.com>
maxulysse and others added 2 commits February 13, 2026 10:38
Co-authored-by: Matthias Hörtenhuber <mashehu@users.noreply.github.com>
@maxulysse maxulysse requested a review from mashehu February 13, 2026 09:44
@maxulysse maxulysse changed the title add prek and gha for validation FEAT: add prek,schema validation and GHA Feb 13, 2026
@lukfor
Copy link
Collaborator

lukfor commented Feb 13, 2026

wow, great idea! thank you guys!

@maxulysse
Copy link
Collaborator Author

wow, great idea! thank you guys!

I made a mistake, I want to make sure it won't happen again 😆

@lukfor
Copy link
Collaborator

lukfor commented Feb 13, 2026

wow, great idea! thank you guys!

I made a mistake, I want to make sure it won't happen again 😆

that’s happened to me more than once as well :)

Co-authored-by: Matthias Hörtenhuber <mashehu@users.noreply.github.com>
@maxulysse maxulysse merged commit e9b7586 into askimed:main Feb 13, 2026
1 check passed
@maxulysse maxulysse deleted the prek branch February 13, 2026 10:09
@lukfor
Copy link
Collaborator

lukfor commented Feb 13, 2026

Thank you guys! I really hope I will have more time soon to help out more actively.

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.

4 participants