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

fix prettier config #250

Merged
merged 6 commits into from
Jan 15, 2024
Merged

fix prettier config #250

merged 6 commits into from
Jan 15, 2024

Conversation

anglepoised
Copy link
Owner

@anglepoised anglepoised commented Jan 11, 2024

Had installed prettier astro plugin but wasn't actually using it.

Still could figure out why prettier refuses to format the generated HTML. Do it, or just go with the generated and minified source and strip out the postbuild. That is probably easier (had originally done this to make html lint work more smoothly, but not sure it's necessary).

Copy link

netlify bot commented Jan 11, 2024

Deploy Preview for anglepoised-notes ready!

Name Link
🔨 Latest commit 650e900
🔍 Latest deploy log https://app.netlify.com/sites/anglepoised-notes/deploys/65a5994c0557b20008f3ff1d
😎 Deploy Preview https://deploy-preview-250--anglepoised-notes.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance:
Accessibility:
Best Practices:
SEO:
PWA: -
View the detailed breakdown and full score reports

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

This wasn't working before, because prettier was picking up paths from
`.gitignore`. Setting an explicit ignore file path overrides this
behaviour.
@anglepoised anglepoised marked this pull request as ready for review January 15, 2024 20:55
@anglepoised anglepoised merged commit 4e7116c into main Jan 15, 2024
5 checks passed
@anglepoised anglepoised deleted the fix-prettier-config branch January 15, 2024 20:55
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.

1 participant