Skip to content

Commit

Permalink
Merge branch '1.5.x' into 2.0.x
Browse files Browse the repository at this point in the history
  • Loading branch information
philwebb committed May 22, 2018
2 parents c1236ed + 9f4f229 commit 6d8b471
Show file tree
Hide file tree
Showing 7 changed files with 27 additions and 19 deletions.
12 changes: 6 additions & 6 deletions .github/ISSUE_TEMPLATE.md
Original file line number Diff line number Diff line change
@@ -1,20 +1,20 @@
<!--
Thanks for raising a Spring Boot issue. What sort of issue are you raising?
Question
❓Question
Please ask questions about how to use something, or to understand why something isn't
working as you expect it to, on Stack Overflow using the spring-boot tag.
Bug report
🐞 Bug report
Please provide details of the problem, including the version of Spring Boot that you
are using. If possible, please provide a test case or sample application that reproduces
the problem. This makes it much easier for us to diagnose the problem and to verify that
we have fixed it.
Enhancement
🚨 Security Vulnerability
STOP!! Please don't raise security vulnerabilities here. Head over to https://pivotal.io/security to learn how to disclose them responsibly.
🎁 Enhancement
Please start by describing the problem that you are trying to solve. There may already
be a solution, or there may be a way to solve it that you hadn't considered.
-->
-->
26 changes: 17 additions & 9 deletions CONTRIBUTING.adoc
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ unacceptable behavior to [email protected].



== Using GitHub issues
== Using GitHub Issues
We use GitHub issues to track bugs and enhancements. If you have a general usage question
please ask on https://stackoverflow.com[Stack Overflow]. The Spring Boot team and the
broader community monitor the https://stackoverflow.com/tags/spring-boot[`spring-boot`]
Expand All @@ -25,6 +25,14 @@ problem.



== Reporting Security Vulnerabilities
If you think you have found a security vulnerability in Spring Boot please *DO NOT*
disclose it publicly until we've had a chance to fix it. Please don't report security
vulnerabilities using GitHub issues, instead head over to https://pivotal.io/security and
learn how to disclose them responsibly.



== Sign the Contributor License Agreement
Before we accept a non-trivial patch or pull request we will need you to
https://cla.pivotal.io/sign/spring[sign the Contributor License Agreement].
Expand Down Expand Up @@ -62,7 +70,7 @@ added after the original pull request but before a merge.



== Working with the code
== Working with the Code
If you don't have an IDE preference we would recommend that you use
https://spring.io/tools/sts[Spring Tools Suite] or
http://eclipse.org[Eclipse] when working with the code. We use the
Expand All @@ -71,13 +79,13 @@ should also work without issue.



=== Building from source
=== Building from Source
To build the source you will need to install
https://maven.apache.org/run-maven/index.html[Apache Maven] v3.2.3 or above and JDK 1.8.



==== Default build
==== Default Build
The project can be built from the root directory using the standard maven command:

[indent=0]
Expand Down Expand Up @@ -125,7 +133,7 @@ can run this from the top-level directory:



=== Importing into eclipse
=== Importing into Eclipse
You can import the Spring Boot code into any Eclipse Mars based distribution. The easiest
way to setup a new environment is to use the Eclipse Installer with the provided
`.setup` file.
Expand Down Expand Up @@ -156,7 +164,7 @@ easier to navigate.



==== Manual installation with m2eclipse
==== Manual Installation with M2Eclipse
If you prefer to install Eclipse yourself we recommend that you use the
http://eclipse.org/m2e/[M2Eclipse] eclipse plugin. If you don't already have m2eclipse
installed it is available from the "Eclipse marketplace".
Expand All @@ -183,7 +191,7 @@ need to import the root `spring-boot` pom and the `spring-boot-samples` pom sepa



==== Importing into eclipse without m2eclipse
==== Importing into Eclipse without M2Eclipse
If you prefer not to use m2eclipse you can generate eclipse project metadata using the
following command:

Expand All @@ -197,12 +205,12 @@ from the `file` menu.



=== Importing into other IDEs
=== Importing into Other IDEs
Maven is well supported by most Java IDEs. Refer to your vendor documentation.



== Integration tests
== Integration Tests
The sample applications are used as integration tests during the build (when you
`mvn install`). Due to the fact that they make use of the `spring-boot-maven-plugin`
they cannot be called directly, and so instead are launched via the
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,6 @@
@DirtiesContext
public class JerseyAutoConfigurationCustomApplicationTests {


@Autowired
private TestRestTemplate restTemplate;

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -402,7 +402,6 @@ private boolean hasLombokPublicAccessor(VariableElement field, TypeElement eleme
return hasAnnotation(element, LOMBOK_DATA_ANNOTATION);
}


private boolean isAccessLevelPublic(AnnotationMirror lombokAnnotation) {
Map<String, Object> values = getAnnotationElementValues(lombokAnnotation);
Object value = values.get("value");
Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/*
* Copyright 2012-2017 the original author or authors.
* Copyright 2012-2018 the original author or authors.
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
Expand Down Expand Up @@ -31,6 +31,7 @@
*/
@Data
@ConfigurationProperties(prefix = "accesslevel.overwrite.data")
@SuppressWarnings("unused")
public class LombokAccessLevelOverwriteDataProperties {

private String name0;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,7 @@
@Getter(AccessLevel.PUBLIC)
@Setter(AccessLevel.PUBLIC)
@ConfigurationProperties(prefix = "accesslevel.overwrite.explicit")
@SuppressWarnings("unused")
public class LombokAccessLevelOverwriteExplicitProperties {

private String name0;
Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/*
* Copyright 2012-2017 the original author or authors.
* Copyright 2012-2018 the original author or authors.
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
Expand Down

0 comments on commit 6d8b471

Please sign in to comment.