Merge remote-tracking branch 'origin' into cayla/fix-linting-errors

Author: Cayla Hamann

.github/workflows/validate-pr.yml

@@ -75,3 +75,29 @@ jobs:
 
       - name: Jest test
         run: yarn test --passWithNoTests
+
+  lint:
+    name: Run Eslint
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v2
+
+      - name: Setup Node.js
+        uses: actions/setup-node@v2
+        with:
+          node-version: 12
+
+      - name: Cache dependencies
+        id: yarn-cache
+        uses: actions/cache@v2
+        with:
+          path: '**/node_modules'
+          key: ${{ runner.os }}-node-modules-${{ hashFiles('**/yarn.lock') }}
+
+      - name: Install dependencies
+        if: steps.yarn-cache.outputs.cache-hit != 'true'
+        run: yarn install --frozen-lockfile
+
+      - name: Run Eslint
+        run: yarn lint

scripts/actions/add-files-to-translation-queue.js

@@ -14,48 +14,61 @@ const { prop } = require('../utils/functional');
  * @returns {Promise<Object<string, string[]>>} An updated queue.
  */
 const getUpdatedQueue = async (url, queue) => {
-  const resp = await fetch(url);
-  const files = await resp.json();
+  try {
+    const resp = await fetch(url);
+    const files = await resp.json();
 
-  const mdxFiles = files
-    .filter((file) => path.extname(file.filename) === '.mdx')
-    .reduce((files, file) => {
-      const contents = fs.readFileSync(path.join(process.cwd(), file.filename));
-      const { data } = frontmatter(contents);
+    const mdxFiles = files
+      ? files
+          .filter((file) => path.extname(file.filename) === '.mdx')
+          .reduce((files, file) => {
+            const contents = fs.readFileSync(
+              path.join(process.cwd(), file.filename)
+            );
+            const { data } = frontmatter(contents);
 
-      return data.translate && data.translate.length
-        ? [...files, { ...file, locales: data.translate }]
-        : files;
-    }, []);
+            return data.translate && data.translate.length
+              ? [...files, { ...file, locales: data.translate }]
+              : files;
+          }, [])
+      : [];
 
-  const addedMdxFiles = mdxFiles
-    .filter((f) => f.status !== 'removed')
-    .reduce((files, file) => {
-      return file.locales.reduce(
-        (acc, locale) => ({
+    const addedMdxFiles = mdxFiles
+      .filter((f) => f.status !== 'removed')
+      .reduce((files, file) => {
+        return file.locales.reduce(
+          (acc, locale) => ({
+            ...acc,
+            [locale]: [...(acc[locale] || []), file.filename],
+          }),
+          files
+        );
+      }, {});
+
+    const removedMdxFileNames = mdxFiles
+      .filter((f) => f.status === 'removed')
+      .map(prop('filename'));
+
+    const queueFiles =
+      Object.entries(queue).length === 0 ? Object.entries(queue) : [];
+
+    return queueFiles
+      .map(([locale, files]) => [
+        locale,
+        files ? files.filter((f) => !removedMdxFileNames.includes(f)) : [],
+      ])
+      .reduce(
+        (acc, [locale, filenames]) => ({
           ...acc,
-          [locale]: [...(acc[locale] || []), file.filename],
+          [locale]: filenames.concat(acc[locale] || []),
         }),
-        files
+        addedMdxFiles
       );
-    }, {});
-
-  const removedMdxFileNames = mdxFiles
-    .filter((f) => f.status === 'removed')
-    .map(prop('filename'));
-
-  return Object.entries(queue)
-    .map(([locale, files]) => [
-      locale,
-      files.filter((f) => !removedMdxFileNames.includes(f)),
-    ])
-    .reduce(
-      (acc, [locale, filenames]) => ({
-        ...acc,
-        [locale]: filenames.concat(acc[locale] || []),
-      }),
-      addedMdxFiles
-    );
+  } catch (error) {
+    console.log(`[!] Unable to get updated queue`);
+    console.log(error);
+    process.exit(1);
+  }
 };
 
 /** Entrypoint. */

src/components/TableOfContents.js

@@ -30,46 +30,14 @@ const TableOfContents = ({ page }) => {
       });
   }, [mdxAST]);
 
-  const raf = useRef();
-  const navRef = useRef();
-  const activeRef = useRef();
   const headingIds = useMemo(() => headings.map(prop('id')), [headings]);
   const activeHash = useActiveHash(headingIds);
-  const previousActiveHash = usePrevious(activeHash);
-  const changedActiveHash = activeHash !== previousActiveHash;
-
   const isMobileScreen = useMedia('(max-width: 1240px)');
 
-  useEffect(() => {
-    if (!activeRef.current) {
-      return;
-    }
-
-    const navRect = navRef.current.getBoundingClientRect();
-    const activeElementRect = activeRef.current.getBoundingClientRect();
-
-    const scrollTop = activeElementRect.top - navRect.top;
-    const offset = activeRef.current.offsetTop - navRef.current.offsetTop;
-    const bottom = scrollTop + activeElementRect.height;
-    const isVisible = bottom <= navRect.height && scrollTop > 0;
-
-    if (!isVisible) {
-      cancelAnimationFrame(raf.current);
-
-      raf.current = requestAnimationFrame(() => {
-        navRef.current.scrollTo({
-          top: offset - navRect.height / 2,
-          behavior: 'smooth',
-        });
-      });
-    }
-  }, [changedActiveHash]);
-
   return headings.length === 0 ? null : (
     <PageTools.Section>
       <PageTools.Title>On this page</PageTools.Title>
       <nav
-        ref={navRef}
         css={css`
           max-height: 60vh;
           overflow-y: auto;
@@ -88,7 +56,6 @@ const TableOfContents = ({ page }) => {
             return (
               <li key={id}>
                 <a
-                  ref={isActive ? activeRef : null}
                   href={`#${id}`}
                   className={isActive && !isMobileScreen ? 'active' : null}
                   css={css`
@@ -112,7 +79,7 @@ const TableOfContents = ({ page }) => {
                   `}
                 >
                   <Icon
-                    name={Icon.TYPE.ARROW_LEFT}
+                    name="fe-arrow-left"
                     css={css`
                       display: ${isActive && !isMobileScreen
                         ? 'inline-block'

src/content/docs/serverless-function-monitoring/aws-lambda-monitoring/enable-lambda-monitoring/enable-serverless-monitoring-aws-lambda.mdx

@@ -14,8 +14,6 @@ redirects:
   - /docs/serverless-function-monitoring/aws-lambda-monitoring/get-started/enable-serverless-monitoring-aws-lambda
 ---
 
-This is a test
-
 <Callout variant="tip">
   The preferred way of enabling AWS Lambda monitoring with New Relic is via our
   Lambda layer. For more information, see [Enable serverless monitoring using