chore(gatsby,gatsby-link): Export Link as named export (#36888)

gatsbyjs · Oct 27, 2022 · 5a88dbb · 5a88dbb
1 parent 4fadb28
commit 5a88dbb
Show file tree

Hide file tree

Showing 7 changed files with 10 additions and 28 deletions.
diff --git a/packages/gatsby-link/index.d.ts b/packages/gatsby-link/index.d.ts
@@ -24,7 +24,7 @@ export interface GatsbyLinkProps<TState> extends LinkProps<TState> {
  * This component is intended _only_ for links to pages handled by Gatsby. For links to pages on other
  * domains or pages on the same domain not handled by the current Gatsby site, use the normal `<a>` element.
  */
-export default class GatsbyLink<TState> extends React.Component<
+export class GatsbyLink<TState> extends React.Component<
   GatsbyLinkProps<TState>,
   any
 > {}

diff --git a/packages/gatsby-link/package.json b/packages/gatsby-link/package.json
@@ -15,13 +15,9 @@
   ],
   "sideEffects": false,
   "scripts": {
-    "build": "npm-run-all --npm-path npm -s build:cjs build:esm",
+    "build": "microbundle -i src/index.js -f cjs,modern --jsx React.createElement --generateTypes false",
     "postbuild": "prepend-directive --files=dist/index.js,dist/index.modern.mjs --directive=\"use client\"",
-    "build:cjs": "microbundle -f cjs --jsx React.createElement --generateTypes false  -i src/index-cjs.js -o dist/index.js",
-    "build:esm": "microbundle -f modern --jsx React.createElement --generateTypes false -o dist/index.mjs",
-    "watch": "npm-run-all --npm-path npm -p watch:cjs watch:esm",
-    "watch:cjs": "npm run build:cjs watch --no-compress",
-    "watch:esm": "npm run build:esm watch --no-compress",
+    "watch": "npm run build watch --no-compress",
     "prepare": "cross-env NODE_ENV=production npm run clean && npm run build",
     "clean": "del-cli dist/*"
   },
@@ -35,7 +31,6 @@
     "cross-env": "^7.0.3",
     "del-cli": "^5.0.0",
     "microbundle": "^0.15.1",
-    "npm-run-all": "^4.1.5",
     "prepend-directive": "^1.0.3"
   },
   "peerDependencies": {

diff --git a/packages/gatsby-link/src/__tests__/index.js b/packages/gatsby-link/src/__tests__/index.js
@@ -9,7 +9,7 @@ import {
   createHistory,
   LocationProvider,
 } from "@reach/router"
-import Link, { navigate, withPrefix, withAssetPrefix } from "../"
+import { Link, navigate, withPrefix, withAssetPrefix } from "../"
 
 beforeEach(() => {
   global.__BASE_PATH__ = ``

diff --git a/packages/gatsby-link/src/index-cjs.js b/packages/gatsby-link/src/index-cjs.js
diff --git a/packages/gatsby-link/src/index.js b/packages/gatsby-link/src/index.js
@@ -1,6 +1,6 @@
 import PropTypes from "prop-types"
 import React from "react"
-import { Link, Location } from "@gatsbyjs/reach-router"
+import { Link as ReachRouterLink, Location } from "@gatsbyjs/reach-router"
 import { parsePath } from "./parse-path"
 import { isLocalLink } from "./is-local-link"
 import { rewriteLinkPath } from "./rewrite-link-path"
@@ -162,7 +162,7 @@ class GatsbyLink extends React.Component {
     }
 
     return (
-      <Link
+      <ReachRouterLink
         to={prefixedTo}
         state={state}
         getProps={getProps}
@@ -220,7 +220,7 @@ GatsbyLink.propTypes = {
   state: PropTypes.object,
 }
 
-export default React.forwardRef((props, ref) => (
+export const Link = React.forwardRef((props, ref) => (
   <GatsbyLinkLocationWrapper innerRef={ref} {...props} />
 ))
 

diff --git a/packages/gatsby/cache-dir/gatsby-browser-entry.js b/packages/gatsby/cache-dir/gatsby-browser-entry.js
@@ -14,7 +14,7 @@ function graphql() {
 export { default as PageRenderer } from "./public-page-renderer"
 export { useScrollRestoration } from "gatsby-react-router-scroll"
 export {
-  default as Link,
+  Link,
   withPrefix,
   withAssetPrefix,
   navigate,

diff --git a/packages/gatsby/index.d.ts b/packages/gatsby/index.d.ts
@@ -23,7 +23,7 @@ export type AvailableFeatures =
   | "content-file-path"
 
 export {
-  default as Link,
+  Link,
   GatsbyLinkProps,
   navigate,
   withPrefix,
@@ -454,10 +454,7 @@ export interface GatsbyNode<
    * @example
    * exports.shouldOnCreateNode = ({node}, pluginOptions) => node.internal.type === 'Image'
    */
-  shouldOnCreateNode?(
-    args: { node: TNode },
-    options: PluginOptions
-  ): boolean
+  shouldOnCreateNode?(args: { node: TNode }, options: PluginOptions): boolean
 
   /**
    * Called when a new page is created. This extension API is useful