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.

Sign up for GitHub

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

Jump to bottom

Omstrukturering og forbedring av kvalitetskontroll #166

Merged
merged 16 commits into from
Oct 5, 2023
Merged

Omstrukturering og forbedring av kvalitetskontroll #166

merged 16 commits into from
Oct 5, 2023

Conversation

fredrik-bakke
Copy link
Collaborator

@fredrik-bakke fredrik-bakke commented Oct 4, 2023 •
edited
Loading

Her kommer noen nye forbedringer til kvalitetskontrollen.

Sammendrag

  • Detekter duplikate oversettelser hvis det er overlapp i termer i alle kolonner. Det er mulig denne regelen er litt for grådig, men jeg klarer ikke komme på et scenarie hvor den feilaktig ville detektert to oversettelser som duplikater av hverandre.
    • Fiks alle forekomster av dette.
  • Detekter ufullstendige oversettelser (0 forekomster). Dette er en ekstensjon av koden som tidligere lette etter tomme linjer.
  • Endre navn på shell skript til kjor_sjekker.sh, og python skript til valider_termtabell.py.
  • Flytt alt av kvalitetskontroll relatert til termtabellen inn i python skriptet.
    • Fjerner avhengigheten til csvlint.
    • Legger grunnarbeid for automatisk sjekk av kommentarformat.
  • Legg til standard .gitignore fil.

@fredrik-bakke
forbedre robusthet og lesbarhet i tester.sh
1e37216 
- Legger til konstanter for feilkoder for å forbedre lesbarheten og vedlikeholdbarheten.
- Inkluderer en funksjon for å sjekke om nødvendige verktøy som csvlint og Python3 er installert.
- Legger til en sjekk for å se om databasefilen eksisterer før skriptet kjøres.
- Qvoterer variabler for å gjøre skriptet mer robust mot spesielle tegn i strenger.
@fredrik-bakke
legg til standard .gitignore
24c7dc7
@fredrik-bakke
alle variabler engelsk tester.sh
e2f40e3
@fredrik-bakke
stilér test.py
aa2d9e5
@fredrik-bakke
navnbytter for skript
f1d8139
@fredrik-bakke
flytt alle termtabell sjekker til python skript
8e1f444
@fredrik-bakke
autodetekter flere duplikate oversettelser
025040e
@fredrik-bakke
fiks detekterte duplikater
1de8a3a
@fredrik-bakke fredrik-bakke added the funksjon Forslag eller forespørsel omhandlende funksjonalitet label Oct 4, 2023
@fredrik-bakke fredrik-bakke changed the title Enda en omstrukturering av kvalitetskontroll Omstrukturering av kvalitetskontroll Oct 4, 2023
@fredrik-bakke fredrik-bakke marked this pull request as draft October 4, 2023 19:05
@fredrik-bakke fredrik-bakke marked this pull request as ready for review October 4, 2023 19:15
@fredrik-bakke fredrik-bakke changed the title Omstrukturering av kvalitetskontroll Omstrukturering og forbedring av kvalitetskontroll Oct 5, 2023
@fredrik-bakke
Copy link
Collaborator Author

Jeg "merger" denne innhentingsforespørselen nå, i påvente av at en eventuell ekstern vurdering vil ta en stund. Gjerne kommenter på denne senere dersom dere har noen bemerkninger!

@fredrik-bakke fredrik-bakke merged commit 76641fe into jfremstad:master Oct 5, 2023
1 check passed
@fredrik-bakke fredrik-bakke deleted the tester branch October 5, 2023 12:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jfremstad jfremstad Awaiting requested review from jfremstad jfremstad is a code owner

@johahaug johahaug Awaiting requested review from johahaug johahaug is a code owner

@bervinator bervinator Awaiting requested review from bervinator bervinator is a code owner

@simonfoldvik simonfoldvik Awaiting requested review from simonfoldvik simonfoldvik is a code owner

Assignees
No one assigned
Labels
funksjon Forslag eller forespørsel omhandlende funksjonalitet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@fredrik-bakke