Merge pull request graphql#316 from wincent/glh/clean

Fix up React warnings and invalid markup

Author: wincent

site/_core/DocsSidebar.js

@@ -12,7 +12,11 @@ import { toSlug } from './Header';
 export default ({ site, page, firstURL }) =>
   <div className="nav-docs">
     {getCategories(site, page.dir, firstURL).map(category =>
-      <SidebarForCategory pageID={page.id} category={category} />
+      <SidebarForCategory
+        category={category}
+        key={category.name}
+        pageID={page.id}
+      />
     )}
   </div>
 

site/community/Community-Resources.md

@@ -30,7 +30,7 @@ Here are some helpful accounts to follow:
 
 ## IRC
 
-Chat with GraphQL developers on IRC at the [https://freenode.net/](freenode) server on the #graphql channel.
+Chat with GraphQL developers on IRC at the [freenode](https://freenode.net/) server on the #graphql channel.
 
 ## Slack & Discord
 

site/graphql-js/Tutorial-BasicTypes.md

@@ -53,6 +53,6 @@ app.listen(4000);
 console.log('Running a GraphQL API server at localhost:4000/graphql');
 ```
 
-If you run this code with `node server.js` and browse to [http://localhost:4000/graphql](http://localhost:4000/graphql) you can try out these APIs.
+If you run this code with `node server.js` and browse to http://localhost:4000/graphql you can try out these APIs.
 
 These examples show you how to call APIs that return different types. To send different types of data into an API, you will also need to learn about [passing arguments to a GraphQL API](/graphql-js/passing-arguments/).

site/graphql-js/Tutorial-GraphQLClients.md

@@ -8,7 +8,7 @@ next: /graphql-js/basic-types/
 
 Since a GraphQL API has more underlying structure than a REST API, there are more powerful clients like [Relay](https://facebook.github.io/relay/) which can automatically handle batching, caching, and other features. But you don't need a complex client to call a GraphQL server. With `express-graphql`, you can just send an HTTP POST request to the endpoint you mounted your GraphQL server on, passing the GraphQL query as the `query` field in a JSON payload.
 
-For example, let's say we mounted a GraphQL server on [http://localhost:4000/graphql](http://localhost:4000/graphql) as in the example code for [running an Express GraphQL server](/graphql-js/running-an-express-graphql-server/), and we want to send the GraphQL query `{ hello }`. We can do this from the command line with `curl`. If you paste this into a terminal:
+For example, let's say we mounted a GraphQL server on http://localhost:4000/graphql as in the example code for [running an Express GraphQL server](/graphql-js/running-an-express-graphql-server/), and we want to send the GraphQL query `{ hello }`. We can do this from the command line with `curl`. If you paste this into a terminal:
 
 ```bash
 curl -X POST \
@@ -23,7 +23,7 @@ You should see the output returned as JSON:
 {"data":{"hello":"Hello world!"}}
 ```
 
-It's also simple to send GraphQL from the browser. Open up [http://localhost:4000](http://localhost:4000/), open a developer console, and paste in:
+It's also simple to send GraphQL from the browser. Open up http://localhost:4000, open a developer console, and paste in:
 
 ```javascript
 var xhr = new XMLHttpRequest();

site/graphql-js/Tutorial-ObjectTypes.md

@@ -137,7 +137,7 @@ When you issue a GraphQL query against an API that returns object types, you can
 }
 ```
 
-If you run this code with `node server.js` and browse to [http://localhost:4000/graphql](http://localhost:4000/graphql) you can try out these APIs with GraphiQL.
+If you run this code with `node server.js` and browse to http://localhost:4000/graphql you can try out these APIs with GraphiQL.
 
 This way of defining object types often provides advantages over a traditional REST API. Instead of doing one API request to get basic information about an object, and then multiple subsequent API requests to find out more information about that object, you can get all of that information in one API request. That saves bandwidth, makes your app run faster, and simplifies your client-side logic.
 

site/graphql-js/Tutorial-PassingArguments.md

@@ -97,7 +97,7 @@ When you call this API, you have to pass each argument by name. So for the serve
 }
 ```
 
-If you run this code with `node server.js` and browse to [http://localhost:4000/graphql](http://localhost:4000/graphql) you can try out this API.
+If you run this code with `node server.js` and browse to http://localhost:4000/graphql you can try out this API.
 
 When you're passing arguments in code, it's generally better to avoid constructing the whole query string yourself. Instead, you can use `$` syntax to define variables in your query, and pass the variables as a separate map.
 

site/index.html.js

@@ -1,8 +1,6 @@
 var React = require('react');
 var Site = require('./_core/Site');
-var SiteData = require('./_core/SiteData');
 var HeaderLinks = require('./_core/HeaderLinks');
-var Marked = require('./_core/Marked');
 var Prism = require('./_core/Prism');
 var Search = require('./_core/Search');