code review fixes

Author: yarinvak

README.md

@@ -343,10 +343,10 @@ The name of the directive will be taken from the ``@GraphQLName`` annotation (if
 The description of the directive will be taken from the ``@GraphQLDescription`` annotation's value.
 The valid locations of the directive (locations which the directive can be applied on) will be taken from ``@DirectiveLocations``.
 The arguments of the directive will be taken from the fields defined in the class - notice that you can only use primitive types as arguments of a directive.
-For example, we defined an ``isActive`` field - which is boolean, and it's default value is true. That's how the argument of the directive will be defined.
+For example, we defined an ``isActive`` field - which is boolean, and its default value is true. That's how the argument of the directive will be defined.
 You can also use ``@GraphQLName`` and ``@GraphQLDescription`` annotations on the field.
 
-After you created the class, you will be able to create the ``GraphQLDirective`` object with the following code:
+After you created the class, you will be able to create the ``GraphQLDirective`` object using the following code:
 ```java
 GraphQLAnnotations.directive(UpperDirective.class);
 ```
@@ -371,7 +371,7 @@ public class UpperWiring implements AnnotationsDirectiveWiring {
         }
     }
 ```
-This class turns you string field to upper case if the directive argument "isActive" is set to true.
+This class turns your string field to upper case if the directive argument "isActive" is set to true.
 Now, you have to wire the field itself:
 ```java
 @GraphQLField

src/main/java/graphql/annotations/processor/directives/CommonPropertiesCreator.java

@@ -1,12 +1,12 @@
 /**
  * Copyright 2016 Yurii Rashkovskii
- *
+ * <p>
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
- *
- *     http://www.apache.org/licenses/LICENSE-2.0
- *
+ * <p>
+ * http://www.apache.org/licenses/LICENSE-2.0
+ * <p>
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
@@ -33,11 +33,9 @@ public String getName(AnnotatedElement annotatedElement) {
         GraphQLName graphQLNameAnnotation = annotatedElement.getAnnotation(GraphQLName.class);
         if (graphQLNameAnnotation != null) {
             return graphQLNameAnnotation.value();
-        }
-        if (annotatedElement instanceof Class<?>) {
+        } else if (annotatedElement instanceof Class<?>) {
             return ((Class<?>) annotatedElement).getSimpleName();
-        }
-        else if (annotatedElement instanceof Field){
+        } else if (annotatedElement instanceof Field) {
             return ((Field) annotatedElement).getName();
         }
         return null;

src/main/java/graphql/annotations/processor/directives/DirectiveArgumentCreator.java

@@ -1,12 +1,12 @@
 /**
  * Copyright 2016 Yurii Rashkovskii
- *
+ * <p>
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
- *
- *     http://www.apache.org/licenses/LICENSE-2.0
- *
+ * <p>
+ * http://www.apache.org/licenses/LICENSE-2.0
+ * <p>
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
@@ -37,10 +37,10 @@ public DirectiveArgumentCreator(CommonPropertiesCreator commonPropertiesCreator,
 
 
     public GraphQLArgument getArgument(Field field, Class<?> containingClass) {
-        GraphQLArgument.Builder builder = newArgument();
-        builder.name(commonPropertiesCreator.getName(field));
-        builder.description(commonPropertiesCreator.getDescription(field));
-        builder.type(getType(field));
+        GraphQLArgument.Builder builder = newArgument()
+                .name(commonPropertiesCreator.getName(field))
+                .description(commonPropertiesCreator.getDescription(field))
+                .type(getType(field));
         try {
             builder.defaultValue(getDefaultValue(field, containingClass));
         } catch (IllegalAccessException | InstantiationException e) {

src/main/java/graphql/annotations/processor/directives/DirectiveCreator.java

@@ -35,9 +35,9 @@ public DirectiveCreator(DirectiveArgumentCreator directiveArgumentCreator, Commo
     }
 
     public GraphQLDirective getDirective(Class<?> annotatedClass) {
-        GraphQLDirective.Builder builder = newDirective();
-        builder.name(commonPropertiesCreator.getName(annotatedClass));
-        builder.description(commonPropertiesCreator.getDescription(annotatedClass));
+        GraphQLDirective.Builder builder = newDirective()
+        .name(commonPropertiesCreator.getName(annotatedClass))
+        .description(commonPropertiesCreator.getDescription(annotatedClass));
         Introspection.DirectiveLocation[] validLocations = getValidLocations(annotatedClass);
         if (validLocations == null || validLocations.length == 0) {
             throw new GraphQLAnnotationsException("No valid locations defined on directive", null);