Enable prettier rules during PR verification

Author: martinwoodward

.github/workflows/pr-verification.yml

@@ -167,39 +167,38 @@ jobs:
           echo "✅ **Build verification passed!** The site is ready for deployment." >> $GITHUB_STEP_SUMMARY
         working-directory: ${{ env.BUILD_PATH }}
 
-  # Temporarily disabled
-  # lint-and-format:
-  #   name: Code Quality Checks
-  #   runs-on: ubuntu-latest
-  #   if: github.event.pull_request.draft == false
-  #   
-  #   steps:
-  #     - name: Checkout
-  #       uses: actions/checkout@v4
-  #       
-  #     - name: Setup Node
-  #       uses: actions/setup-node@v4
-  #       with:
-  #         node-version: "20"
-  #         # Disable cache since we're manually clearing everything
-  #         
-  #     - name: Install dependencies
-  #       run: |
-  #         # Clear npm cache to avoid platform-specific binary issues
-  #         npm cache clean --force
-  #         # Remove package-lock.json and node_modules to fix Rollup optional dependency issue
-  #         rm -rf package-lock.json node_modules
-  #         # Fresh install
-  #         npm install
-  #       working-directory: ${{ env.BUILD_PATH }}
-  #       
-  #     - name: Run Prettier format check
-  #       run: |
-  #         # Check if code is properly formatted
-  #         npx prettier --check ./src
-  #       working-directory: ${{ env.BUILD_PATH }}
-  #       
-  #     - name: TypeScript type check
-  #       run: |
-  #         npx astro check
-  #       working-directory: ${{ env.BUILD_PATH }}
+  lint-and-format:
+    name: Code Quality Checks
+    runs-on: ubuntu-latest
+    if: github.event.pull_request.draft == false
+    
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+        
+      - name: Setup Node
+        uses: actions/setup-node@v4
+        with:
+          node-version: "20"
+          # Disable cache since we're manually clearing everything
+          
+      - name: Install dependencies
+        run: |
+          # Clear npm cache to avoid platform-specific binary issues
+          npm cache clean --force
+          # Remove package-lock.json and node_modules to fix Rollup optional dependency issue
+          rm -rf package-lock.json node_modules
+          # Fresh install
+          npm install
+        working-directory: ${{ env.BUILD_PATH }}
+        
+      - name: Run Prettier format check
+        run: |
+          # Check if code is properly formatted
+          npx prettier --check ./src
+        working-directory: ${{ env.BUILD_PATH }}
+        
+      - name: TypeScript type check
+        run: |
+          npx astro check
+        working-directory: ${{ env.BUILD_PATH }}

src/components/primer/PrimerHeading.tsx

@@ -1,37 +1,46 @@
-import React from 'react';
-import type { ReactNode } from 'react';
+import React from "react";
+import type { ReactNode } from "react";
 
 interface PrimerHeadingProps {
   level?: 1 | 2 | 3 | 4 | 5 | 6;
-  size?: 'display' | '1' | '2' | '3' | '4' | '5' | '6' | 'subhead-large' | 'subhead-medium';
+  size?:
+    | "display"
+    | "1"
+    | "2"
+    | "3"
+    | "4"
+    | "5"
+    | "6"
+    | "subhead-large"
+    | "subhead-medium";
   children: ReactNode;
   className?: string;
 }
 
-export default function PrimerHeading({ 
-  level = 2, 
-  size = '3', 
-  children, 
-  className = '' 
+export default function PrimerHeading({
+  level = 2,
+  size = "3",
+  children,
+  className = "",
 }: PrimerHeadingProps) {
   // Use dynamic import for client-side rendering
   const [Heading, setHeading] = React.useState<any>(null);
-  
+
   React.useEffect(() => {
     async function loadHeading() {
       try {
-        const { Heading: PrimerHeading } = await import('@primer/react-brand');
+        const { Heading: PrimerHeading } = await import("@primer/react-brand");
         setHeading(() => PrimerHeading);
       } catch (error) {
-        console.error('Failed to load Primer Heading:', error);
+        console.error("Failed to load Primer Heading:", error);
       }
     }
     loadHeading();
   }, []);
 
   if (!Heading) {
     // Fallback to regular heading element
-    const Tag = `h${level}` as 'h1' | 'h2' | 'h3' | 'h4' | 'h5' | 'h6';
+    const Tag = `h${level}` as "h1" | "h2" | "h3" | "h4" | "h5" | "h6";
     return React.createElement(Tag, { className }, children);
   }
 

src/components/primer/PrimerText.tsx

@@ -1,33 +1,44 @@
-import React from 'react';
-import type { ReactNode } from 'react';
+import React from "react";
+import type { ReactNode } from "react";
 
 interface PrimerTextProps {
-  size?: '100' | '200' | '300' | '350' | '400' | '500' | '600' | '700' | '800' | '900' | '1000';
-  weight?: 'light' | 'normal' | 'medium' | 'semibold';
-  variant?: 'default' | 'muted';
+  size?:
+    | "100"
+    | "200"
+    | "300"
+    | "350"
+    | "400"
+    | "500"
+    | "600"
+    | "700"
+    | "800"
+    | "900"
+    | "1000";
+  weight?: "light" | "normal" | "medium" | "semibold";
+  variant?: "default" | "muted";
   children: ReactNode;
   className?: string;
-  as?: 'p' | 'span' | 'div';
+  as?: "p" | "span" | "div";
 }
 
-export default function PrimerText({ 
-  size = '400', 
-  weight = 'normal',
-  variant = 'default',
-  children, 
-  className = '',
-  as = 'p'
+export default function PrimerText({
+  size = "400",
+  weight = "normal",
+  variant = "default",
+  children,
+  className = "",
+  as = "p",
 }: PrimerTextProps) {
   // Use dynamic import for client-side rendering
   const [Text, setText] = React.useState<any>(null);
-  
+
   React.useEffect(() => {
     async function loadText() {
       try {
-        const { Text: PrimerText } = await import('@primer/react-brand');
+        const { Text: PrimerText } = await import("@primer/react-brand");
         setText(() => PrimerText);
       } catch (error) {
-        console.error('Failed to load Primer Text:', error);
+        console.error("Failed to load Primer Text:", error);
       }
     }
     loadText();
@@ -39,10 +50,10 @@ export default function PrimerText({
   }
 
   return (
-    <Text 
-      size={size} 
-      weight={weight} 
-      variant={variant} 
+    <Text
+      size={size}
+      weight={weight}
+      variant={variant}
       as={as}
       className={className}
     >

src/components/primer/index.ts

@@ -1,2 +1,2 @@
-export { default as PrimerHeading } from './PrimerHeading';
-export { default as PrimerText } from './PrimerText';
+export { default as PrimerHeading } from "./PrimerHeading";
+export { default as PrimerText } from "./PrimerText";

src/config/menu.json

@@ -51,7 +51,7 @@
           "name": "Linkedin",
           "url": "https://www.linkedin.com/in/martinwoodward"
         },
-                {
+        {
           "name": "Twitter",
           "url": "https://x.com/martinwoodward"
         },

src/content/pages/elements.mdx

@@ -4,6 +4,7 @@ meta_title: ""
 description: "this is meta description"
 draft: false
 ---
+
 import { PrimerHeading, PrimerText } from "../../components/primer";
 import Button from "../../layouts/shortcodes/Button.tsx";
 import Notice from "../../layouts/shortcodes/Notice.tsx";
@@ -39,7 +40,8 @@ Here is an example of headings. You can use this heading by the following markdo
 </PrimerText>
 
 <PrimerText size="400" weight="normal">
-  This is normal text using Primer React components. The size is 400 which is a good default for body text.
+  This is normal text using Primer React components. The size is 400 which is a
+  good default for body text.
 </PrimerText>
 
 <PrimerText size="300" weight="light" variant="muted">

src/content/pages/rss.md

@@ -12,6 +12,7 @@ Stay updated with the latest posts by subscribing to our RSS feeds. Choose the f
 ## Main Feeds
 
 ### [All Posts RSS Feed](/rss.xml)
+
 Subscribe to get all published blog posts delivered to your RSS reader.
 
 # Category Feeds
@@ -21,7 +22,7 @@ You can also subscribe to specific category feeds to get posts only from topics
 - Photography: `/rss/photography.xml`
 - Test: `/rss/test.xml`
 
-*Note: Category feeds are automatically generated based on the categories used in blog posts. Replace spaces with hyphens and use lowercase letters in the URL.*
+_Note: Category feeds are automatically generated based on the categories used in blog posts. Replace spaces with hyphens and use lowercase letters in the URL._
 
 ## How to Use RSS Feeds
 

src/content/post/2000/01/2000-01-12-ba-miles.md

@@ -14,4 +14,5 @@ tags: ["blog", "archive"]
 type: "regular" # available type (regular or featured)
 draft: false
 ---
-This site is noteworthy as it was the first live deployment of a jsp page on the BA site - before then it was all jhtml. Other than that not really proud of this one as I had 4 days to get it designed, written, tested and live due to an upcoming marketing campaign that this site was supporting.
+
+This site is noteworthy as it was the first live deployment of a jsp page on the BA site - before then it was all jhtml. Other than that not really proud of this one as I had 4 days to get it designed, written, tested and live due to an upcoming marketing campaign that this site was supporting.

src/content/post/2000/04/2000-04-12-online-checkin.md

@@ -14,4 +14,5 @@ tags: ["blog", "archive"]
 type: "regular" # available type (regular or featured)
 draft: false
 ---
-Member of the team who produced the pioneering online check-in site for BA.   Full J2EE n-tier application, however the best bit is the "little man" interface where you can click on a seat to select which seat you want and the little man moves.  A shortcut feature is that you can actually type the number in an it moves the little man.  This style of interface has been much re-used on other systems since it was done.
+
+Member of the team who produced the pioneering online check-in site for BA. Full J2EE n-tier application, however the best bit is the "little man" interface where you can click on a seat to select which seat you want and the little man moves. A shortcut feature is that you can actually type the number in an it moves the little man. This style of interface has been much re-used on other systems since it was done.

src/content/post/2000/05/2000-05-20-html-editor.md

@@ -14,4 +14,5 @@ tags: ["blog", "archive"]
 type: "regular" # available type (regular or featured)
 draft: false
 ---
-Demonstration of the [CONTENTEDITABLE](http://msdn.microsoft.com/workshop/author/dhtml/reference/properties/contentEditable.asp) attribute that was introduced in IE 5.5 [Show Example](/sandbox/htmledit.html).
+
+Demonstration of the [CONTENTEDITABLE](http://msdn.microsoft.com/workshop/author/dhtml/reference/properties/contentEditable.asp) attribute that was introduced in IE 5.5 [Show Example](/sandbox/htmledit.html).