Fix code example renders (#1969)

* removes deprecated Octicons export from docs scope

* undo the change of  order in exports

* adds comment about why we do not export octicons-react default export

* Update docs/src/@primer/gatsby-theme-doctocat/live-code-scope.js

* Update export

Co-authored-by: Cole Bemis <colebemis@github.com>

Author: 2 people

docs/src/@primer/gatsby-theme-doctocat/live-code-scope.js

@@ -12,11 +12,14 @@ const ReactRouterLink = ({to, ...props}) => {
   return <a href={to} {...props} />
 }
 
+// Exclude octicons-react's default export because it's deprecated
+const {default: _, ...octiconComponents} = octicons
+
 export default function resolveScope(metastring) {
   return {
     ...doctocatComponents,
     ...primerComponents,
-    ...octicons,
+    ...octiconComponents,
     ...(metastring.includes('drafts') ? drafts : {}),
     ...(metastring.includes('deprecated') ? deprecated : {}),
     ReactRouterLink,