Reduce prettier overrides (#30)

## Ticket

#24 

## Changes
This PR removed some non-default settings for prettier and then also
made the formatting changes associated with that setting change. The
setting change and the format change are in separate commits, in case
that makes it easier to review.

## Context for reviewers
All changes should be non-functional, so just visual confirmation that
this looks right is probably all that's needed. Rationale for the change
is provided in the issue (#24).

## Testing
Non-functional changes, so little testing was needed, but I still ran
storybook and the next.js app to make sure those still work, and they
did.

Author: jimmyfagan

app/.prettierrc.json

@@ -1,7 +1,5 @@
 {
   "importOrder": ["^components/(.*)$", "^[./]"],
   "importOrderSeparation": true,
-  "importOrderSortSpecifiers": true,
-  "semi": false,
-  "singleQuote": true
+  "importOrderSortSpecifiers": true
 }

app/.storybook/i18next.js

@@ -1,30 +1,30 @@
 // Configure i18next for storybook addon storybook-react-i18next
 // See https://storybook.js.org/addons/storybook-react-i18next
-import i18n from 'i18next'
-import LanguageDetector from 'i18next-browser-languagedetector'
-import Backend from 'i18next-http-backend'
-import { initReactI18next } from 'react-i18next'
+import i18n from "i18next";
+import LanguageDetector from "i18next-browser-languagedetector";
+import Backend from "i18next-http-backend";
+import { initReactI18next } from "react-i18next";
 
-const ns = ['common']
-const supportedLngs = ['en', 'es']
+const ns = ["common"];
+const supportedLngs = ["en", "es"];
 const resources = ns.reduce((acc, n) => {
   supportedLngs.forEach((lng) => {
-    if (!acc[lng]) acc[lng] = {}
+    if (!acc[lng]) acc[lng] = {};
     acc[lng] = {
       ...acc[lng],
       [n]: require(`../public/locales/${lng}/${n}.json`),
-    }
-  })
-  return acc
-}, {})
+    };
+  });
+  return acc;
+}, {});
 
 i18n.use(initReactI18next).use(LanguageDetector).use(Backend).init({
-  lng: 'en',
-  fallbackLng: 'en',
-  defaultNS: 'common',
+  lng: "en",
+  fallbackLng: "en",
+  defaultNS: "common",
   ns,
   supportedLngs,
   resources,
-})
+});
 
-export default i18n
+export default i18n;

app/.storybook/main.js

@@ -4,56 +4,56 @@
  * @see https://storybook.js.org/docs/configurations/default-config/
  */
 
-const nextConfig = require('../next.config')
+const nextConfig = require("../next.config");
 
 module.exports = {
-  stories: ['../stories/**/*.stories.@(mdx|js|jsx|ts|tsx)'],
-  addons: ['@storybook/addon-essentials', 'storybook-react-i18next'],
-  framework: '@storybook/react',
+  stories: ["../stories/**/*.stories.@(mdx|js|jsx|ts|tsx)"],
+  addons: ["@storybook/addon-essentials", "storybook-react-i18next"],
+  framework: "@storybook/react",
   core: {
     // Use webpack5 instead of webpack4.
-    builder: 'webpack5',
+    builder: "webpack5",
     disableTelemetry: true,
   },
   // Tell storybook where to find USWDS static assets
-  staticDirs: ['../public'],
+  staticDirs: ["../public"],
 
   // Configure Storybook's final Webpack configuration in order to re-use the Next.js config/dependencies.
   webpackFinal: (config) => {
     config.module?.rules?.push({
       test: /\.scss$/,
       use: [
-        'style-loader',
-        { loader: 'css-loader', options: { url: false } }, // this mirrors next.js behavior
+        "style-loader",
+        { loader: "css-loader", options: { url: false } }, // this mirrors next.js behavior
         {
           /**
            * Next.js sets this automatically for us, but we need to manually set it here for Storybook.
            * The main thing this enables is autoprefixer, so any experimental CSS properties work.
            */
-          loader: 'postcss-loader',
+          loader: "postcss-loader",
           options: {
             postcssOptions: {
-              plugins: ['postcss-preset-env'],
+              plugins: ["postcss-preset-env"],
             },
           },
         },
         {
-          loader: 'sass-loader',
+          loader: "sass-loader",
           options: {
             sassOptions: nextConfig.sassOptions,
           },
         },
       ],
       exclude: /node_modules/,
-    })
+    });
 
     // Required for i18next.
     config.resolve.fallback = {
       fs: false,
       path: false,
       os: false,
-    }
+    };
 
-    return config
+    return config;
   },
-}
+};

app/.storybook/preview.js

@@ -1,10 +1,10 @@
 // Apply global styling to our stories
-import '../styles/styles.scss'
+import "../styles/styles.scss";
 // Import i18next config.
-import i18n from './i18next.js'
+import i18n from "./i18next.js";
 
 export const parameters = {
-  actions: { argTypesRegex: '^on[A-Z].*' },
+  actions: { argTypesRegex: "^on[A-Z].*" },
   controls: {
     matchers: {
       color: /(background|color)$/i,
@@ -13,9 +13,9 @@ export const parameters = {
   },
   // Configure i18next and locale/dropdown options.
   i18n,
-  locale: 'en',
+  locale: "en",
   locales: {
-    en: 'English',
-    es: 'Español',
+    en: "English",
+    es: "Español",
   },
-}
+};

app/__mocks__/styleMock.js

@@ -1 +1 @@
-module.exports = {}
+module.exports = {};

app/jest.config.js

@@ -1,29 +1,29 @@
 /** @type {import('ts-jest/dist/types').InitialOptionsTsJest} */
 
 module.exports = {
-  preset: 'ts-jest',
-  testEnvironment: 'jsdom',
+  preset: "ts-jest",
+  testEnvironment: "jsdom",
   moduleNameMapper: {
-    '\\.(css|scss)$': '<rootDir>/__mocks__/styleMock.js', // sets up routing to mocks style sheet
-    '^@pages(.*)$': '<rootDir>/pages$1', //allows module imports of page components
+    "\\.(css|scss)$": "<rootDir>/__mocks__/styleMock.js", // sets up routing to mocks style sheet
+    "^@pages(.*)$": "<rootDir>/pages$1", //allows module imports of page components
   },
   setupFilesAfterEnv: [
-    '<rootDir>/tests/jest.setup.js',
-    '<rootDir>/tests/jest-i18n.ts',
+    "<rootDir>/tests/jest.setup.js",
+    "<rootDir>/tests/jest-i18n.ts",
   ],
   transform: {
-    '^.+\\.tsx?$': 'ts-jest',
+    "^.+\\.tsx?$": "ts-jest",
   }, //transfrom typescript files to common js for jest compiler
   transformIgnorePatterns: [
-    '/.next/',
-    '/node_modules/',
-    '^.+\\.module\\.(css|sass|scss)$',
+    "/.next/",
+    "/node_modules/",
+    "^.+\\.module\\.(css|sass|scss)$",
   ],
-  testPathIgnorePatterns: ['<rootDir>/node_modules/'],
-  testRegex: '(/tests/.*(test|spec))\\.[jt]sx?$',
+  testPathIgnorePatterns: ["<rootDir>/node_modules/"],
+  testRegex: "(/tests/.*(test|spec))\\.[jt]sx?$",
   globals: {
-    'ts-jest': {
-      tsconfig: 'tsconfig.ts-jest.json',
+    "ts-jest": {
+      tsconfig: "tsconfig.ts-jest.json",
     },
   },
-}
+};

app/next-i18next.config.js

@@ -1,11 +1,11 @@
 module.exports = {
   i18n: {
-    defaultLocale: 'en',
-    locales: ['en', 'es'],
+    defaultLocale: "en",
+    locales: ["en", "es"],
     react: {
       // Add support for <em>.
       // See https://react.i18next.com/latest/trans-component#using-for-less-than-br-greater-than-and-other-simple-html-elements-in-translations-v-10-4-0
-      transKeepBasicHtmlNodesFor: ['br', 'strong', 'i', 'p', 'em'],
+      transKeepBasicHtmlNodesFor: ["br", "strong", "i", "p", "em"],
     },
   },
-}
+};

app/next.config.js

@@ -1,16 +1,16 @@
 /** @type {import('next').NextConfig} */
 
-const { i18n } = require('./next-i18next.config')
+const { i18n } = require("./next-i18next.config");
 
 const nextConfig = {
   i18n,
   reactStrictMode: true,
   sassOptions: {
     includePaths: [
-      './node_modules/@uswds',
-      './node_modules/@uswds/uswds/packages',
+      "./node_modules/@uswds",
+      "./node_modules/@uswds/uswds/packages",
     ],
   },
-}
+};
 
-module.exports = nextConfig
+module.exports = nextConfig;

app/src/api/hello.ts

@@ -1,13 +1,13 @@
 // Next.js API route support: https://nextjs.org/docs/api-routes/introduction
-import type { NextApiRequest, NextApiResponse } from 'next'
+import type { NextApiRequest, NextApiResponse } from "next";
 
 type Data = {
-  name: string
-}
+  name: string;
+};
 
 export default function handler(
   req: NextApiRequest,
   res: NextApiResponse<Data>
 ) {
-  res.status(200).json({ name: 'John Doe' })
+  res.status(200).json({ name: "John Doe" });
 }

app/src/components/Layout.tsx

@@ -1,24 +1,24 @@
-import { useTranslation } from 'next-i18next'
-import { ReactElement } from 'react'
+import { useTranslation } from "next-i18next";
+import { ReactElement } from "react";
 
 type Props = {
-  children: ReactElement
-}
+  children: ReactElement;
+};
 
 const Layout = ({ children }: Props): ReactElement => {
-  const { t } = useTranslation('common')
+  const { t } = useTranslation("common");
 
   return (
     <div className="container">
       <header className="header">
-        <em>{t('Layout.header')}</em>
+        <em>{t("Layout.header")}</em>
       </header>
       <main className="main">{children}</main>
       <footer className="footer">
-        <em>{t('Layout.footer')}</em>
+        <em>{t("Layout.footer")}</em>
       </footer>
     </div>
-  )
-}
+  );
+};
 
-export default Layout
+export default Layout;