Skip to content

Conversation

@lilstiffy
Copy link
Contributor

make sure to drop your tables before we merge this for convenience :D
i left scan_details as plural because i think it makes sense.

Would be weird in swedish as well, Skandetalj instead of Skandetaljer, but let me know what you think 🫡

@lilstiffy lilstiffy requested a review from pwgit-create July 26, 2025 17:49
@lilstiffy lilstiffy self-assigned this Jul 26, 2025
Copy link
Contributor

@pwgit-create pwgit-create left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bra att ni tog tag i denna ändå , så blir repot fint 👍
Ser bra ut men det är två saker ni behöver fixa innan merge:

  1. Tabell namnet scan_details -> scan_summary = ScanSummary

Detta är så alla tabeller uppfyller singularis. Consistency är viktigare än singularis / pluralis.

  1. Ni behöver göra en PR i File-Integrity_Scanner samtidigt som byter mappnings namnen för tabellerna i Entity klasserna mot de nya.

EDIT: Såg att ni gjorde detta i draft PR branch 45! Det går bra att göra så, men mergen till Master kommer ske innan pwssOrg/File-Integrity-Scanner#45 är färdig efter som det är en såppas stor puck.

Vi kan merga in 45 i master senare - för vill absolut inte att ni ska stressa med denna - då blir den dålig. Fortsätt som ni gör med den pucken så blir det bra @lilstiffy 🥇

@pwgit-create
Copy link
Contributor

make sure to drop your tables before we merge this for convenience :D i left scan_details as plural because i think it makes sense.

Would be weird in swedish as well, Skandetalj instead of Skandetaljer, but let me know what you think 🫡

I've added my thoughts on this topic in the review 🗒️

@lilstiffy
Copy link
Contributor Author

ScanSummary is a great name, renamed the table :D

@pwgit-create
Copy link
Contributor

ScanSummary is a great name, renamed the table :D

😃 💯

Copy link
Contributor

@pwgit-create pwgit-create left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💠

@lilstiffy lilstiffy merged commit 2870941 into master Jul 28, 2025
@lilstiffy lilstiffy deleted the rename-to-tables-to-singular branch July 28, 2025 14:47
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.

3 participants