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

frontend linting #230

Closed
wants to merge 7 commits into from
Closed

frontend linting #230

wants to merge 7 commits into from

Conversation

ArnoutAllaert
Copy link
Contributor

Ik heb frontend linting bij django.yml toegevoegd zodat dit ook gecheckt wordt bij pull requests.

@ArnoutAllaert ArnoutAllaert marked this pull request as draft May 6, 2024 11:59
@ArnoutAllaert
Copy link
Contributor Author

ik ga ook nog de frontend testen aan de actions toevoegen

@ArnoutAllaert
Copy link
Contributor Author

@mathis2003 weet jij hoe we dit kunnen fixen?
image

@mathis2003
Copy link
Contributor

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"

@mathis2003
Copy link
Contributor

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.

eliasnijs
eliasnijs approved these changes May 6, 2024
eliasnijs

This comment was marked as spam.

miboelae

This comment was marked as spam.

@SELab-2 SELab-2 deleted a comment from ArnoutAllaert May 20, 2024
@SELab-2 SELab-2 deleted a comment from Gerwoud May 23, 2024
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.

5 participants