issues
search
ServiceNow
/
Fast-LLM
Accelerating your LLM training to full speed
https://servicenow.github.io/Fast-LLM/
Other
37
stars
5
forks
source link
[feat] Require linting to pass before merging
#54
Open
jlamypoirier
opened
1 day ago
jlamypoirier
commented
1 day ago
🧐 Problem Description
We need to tell every new contributor to setup pre-commit.
💡 Proposed Solution
Add pre-commit requirement to contributing guidelines, maybe also to the PR template.
Add a pre-commit test to CI
Require CI to pass before merging (need to fix the flaky docker build first though)
🧐 Problem Description
We need to tell every new contributor to setup pre-commit.
💡 Proposed Solution