chore: fix prettier, move pages, add husky

[isabellabrookes]

• Added husky for pre-commit hooks
• Updated documentation
• Moved pages to their own subdir _pages/ to clean up root
• Ran prettier on all files
• Added GHA for formatting via prettier on pull-request

[netlify]

✅ Deploy Preview for twc-site-nl ready!

Name	Link
🔨 Latest commit	1f30423
🔍 Latest deploy log	https://app.netlify.com/sites/twc-site-nl/deploys/668d154da63cab0008c639d7
😎 Deploy Preview	https://deploy-preview-14--twc-site-nl.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[thisislawatts]

Praise: Thanks for introducing Prettier @isabellabrookes that is a really positive change ✨

Suggestion (non-blocking): Perhaps it would be worth adopting a GitHub action to lint the changes to avoid regressions being introduced by folks who opt of git hooks

[b-n]

@isabellabrookes do you feel strongly about precommit hooks? I have a preference for merge checks vs. precommit.

Happy to describe the reasons why, but shortcutting the conversation by asking the question first.

I'm happy to make/push a Github Action for prettier as a pre merge check since I'm asking for the alternative.

[isabellabrookes]

@thisislawatts & @b-n I initially had a GHA, but then I dropped it because I thought I was adding too much.

Techincally, the githook is optional. You don't have to run npm install at all. Especially if you have prettier installed globally.

Options:

1. remove pre-commit hook + add gha
2. keep pre-commit hook, redefine the docs to make it optional + add alternative for global prettier
3. number 2 + gha

[b-n]

@isabellabrookes Opt 3 has my preference. LMK if I can help with anything, and much appreciation for this work!

[b-n]

Whilst here I thought I'd check file by file (otherwise I'm a hypocrit at work 😅).

"Request changes" per the GHA discussion.

There is a question around events.[yaml|json] I have 🤔

The other comments are mostly just so I remember to make issues. Found this along the way: https://github.com/adamzapasnik/prettier-plugin-erb (for a new PR though)

[b-n]

Hrmmmm. Site still works without this?

There is an admin page here: https://techwerkers.nl/admin/ and I'm wondering if this file is needed there,

[isabellabrookes]

Honestly, I yolo-ed. It didn't break my the docker env. I don't have access to netlify, so I'm not sure it's something I can check. Can we use the preview: https://deploy-preview-14--twc-site-nl.netlify.app/admin ?

[b-n]

As previous comment too

[isabellabrookes]

@b-n & @thisislawatts

1. fixed the erb liquid issue
2. updated the docs with all the options for pre-commit hooks/prettier - here
3. added a GHA that runs prettier, and commits any changes for you. Tested successfully here - (2261c9a)
4. As for the events.json/yml - I have no clue as I can't access netlify. I can put them back easily enough

[b-n]

One last little bit to discuss with the action. The rest of the changes are ✅

[b-n]