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

Added Virtuoso frame and modules #70

Closed
wants to merge 11 commits into from
Closed

Conversation

kennethdammyr
Copy link
Contributor

No description provided.

@kennethdammyr kennethdammyr requested a review from a team as a code owner June 7, 2024 09:45
@kennethdammyr
Copy link
Contributor Author

Kan noen voksne hjelpe meg å forstå hvorfor formateringsjekken feiler?

@heidisu
Copy link
Member

heidisu commented Jun 7, 2024

Kan noen voksne hjelpe meg å forstå hvorfor formateringsjekken feiler?

Forstår ikke hva som feiler her jeg heller, vanligvis så sier den i fra om hva det er og hvilken fil, så tyder på at det kan være noe annet mer overordnet som egentlig feiler.

Men når jeg først begynte å myse ser jeg at du vil få problemer med test-steget som kommer etter, som sjekker at filnavn matcher modell-navn.

@sjurtf
Copy link
Member

sjurtf commented Jun 7, 2024

pre-commit hooken hos meg endrer på disse to aud-aes3.yaml og aud-rpro.yaml.
Den splitter description feltet i to linjer ettersom det er over ruamel sin default bredde.

Testene feiler også på en del filer.

Vil anbefale å kjøre disse:

$ pre-commit install 
$ pre-commit run --all

@sjurtf sjurtf self-requested a review June 7, 2024 11:51
@kennethdammyr
Copy link
Contributor Author

pre-commit hooken hos meg endrer på disse to aud-aes3.yaml og aud-rpro.yaml. Den splitter description feltet i to linjer ettersom det er over ruamel sin default bredde.

Testene feiler også på en del filer.

Vil anbefale å kjøre disse:

$ pre-commit install 
$ pre-commit run --all

Jeg kjører pre-commit, men den bare toucher en haug med filer, uten å gjøre noe med de aktuelle filene...

@sjurtf sjurtf closed this Jun 7, 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.

4 participants