-
Notifications
You must be signed in to change notification settings - Fork 465
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Sample Quickstarts to project #33
Add Sample Quickstarts to project #33
Conversation
Codecov Report
@@ Coverage Diff @@
## master #33 +/- ##
=========================================
Coverage 24.46% 24.46%
Complexity 1334 1334
=========================================
Files 326 326
Lines 17603 17603
Branches 2691 2691
=========================================
Hits 4307 4307
Misses 12805 12805
Partials 491 491 Continue to review full report at Codecov.
|
<modelVersion>4.0.0</modelVersion> | ||
|
||
<artifactId>jshift-maven-sample-spring-boot</artifactId> | ||
<groupId>io.jshift</groupId> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Replace by valid groupID (also artifcatId is not OK)
<build> | ||
<plugins> | ||
<plugin> | ||
<groupId>io.jshift</groupId> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same. Check all files for "fabric8" or "jshift"
* Contributors: | ||
* Red Hat, Inc. - initial API and implementation | ||
*/ | ||
package io.fabric8.mven.sample.thorntail.rest; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we want to keep fabric8 package names?
<html> | ||
<head> | ||
<meta charset="UTF-8"> | ||
<title>fabric8:Maven Plugin</title> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe use more appropriate values e.g. "Eclipse JKube"
</head> | ||
|
||
<body> | ||
<h2>fabric8 maven plugin rocks!!</h2> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
JKube is trendier!
e8a82b6
to
1cf6dae
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
These are not directly linked to project so not sure whether we should file CQs for these or not.