-
Notifications
You must be signed in to change notification settings - Fork 0
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
frontend linting #230
frontend linting #230
Conversation
ik ga ook nog de frontend testen aan de actions toevoegen |
@mathis2003 weet jij hoe we dit kunnen fixen? |
Dat is raar. Ik zal eens kijken hoeveel ruimte we hebben op de server. Kan je anders proberen verder werken door de github actions te laten runnen op de github servers zelf? Sinds ons project nu publiek staat zouden we toch ongelimiteerd rekentijd moeten hebben als ik mij niet vergis. Het is heel simpel om te doen. Bij het "runs-on" veld in de django.yml zet je gewoon "ubuntu-latest" in plaats van "self-hosted" |
Daarover gesproken... Waarschijnlijk wordt het tijd dat we de django.yml file opsplitsen in meerdere files. Om de tests wat overzichtelijker te houden. Maar dat is bijzaak momenteel. |
Ik heb frontend linting bij django.yml toegevoegd zodat dit ook gecheckt wordt bij pull requests.