Skip to content

Conversation

eric-lee-ltk
Copy link
Contributor

No description provided.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 85.577% when pulling 967958d on eric-lee-ltk:quick-start-sample into 5dd3232 on ServiceComb:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 85.577% when pulling 277c311 on eric-lee-ltk:quick-start-sample into 5dd3232 on ServiceComb:master.

<parent>
<artifactId>bmi</artifactId>
<groupId>io.servicecomb.samples</groupId>
<version>0.0.1-SNAPSHOT</version>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We may use the same version of Java Chassis.


<groupId>io.servicecomb.samples</groupId>
<artifactId>bmi</artifactId>
<version>0.0.1-SNAPSHOT</version>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The same issue here.

<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
<project.reporting.outputEncoding>UTF-8</project.reporting.outputEncoding>
<java.version>1.8</java.version>
<java-chassis.version>0.2.0</java-chassis.version>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Using the project version.

<parent>
<artifactId>bmi</artifactId>
<groupId>io.servicecomb.samples</groupId>
<version>0.0.1-SNAPSHOT</version>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Using the project version.

@@ -0,0 +1,12 @@
zuul:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add the license header.

@@ -0,0 +1,13 @@
<?xml version="1.0" encoding="UTF-8" ?>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add the license header.

@@ -0,0 +1,10 @@
# all interconnected microservices must belong to an application wth the same ID
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add the license header here.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 85.577% when pulling cfde150 on eric-lee-ltk:quick-start-sample into 5dd3232 on ServiceComb:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.03%) to 85.545% when pulling 7edae0c on eric-lee-ltk:quick-start-sample into a837479 on ServiceComb:master.

@WillemJiang WillemJiang merged commit e6e94ae into apache:master Sep 8, 2017
@eric-lee-ltk eric-lee-ltk deleted the quick-start-sample branch September 8, 2017 06:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants