Skip to content

Commit

Permalink
Merge pull request #65 from jGauravGupta/FISH-612
Browse files Browse the repository at this point in the history
FISH-612 Release Payara VSCode extension 1.0.0
  • Loading branch information
jGauravGupta authored Dec 18, 2020
2 parents b28b3e7 + 7b4d23f commit c6abd00
Show file tree
Hide file tree
Showing 3 changed files with 21 additions and 5 deletions.
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
"name": "payara-vscode",
"displayName": "Payara Tools",
"description": "Payara Tools for Visual Studio Code",
"version": "0.0.5-SNAPSHOT",
"version": "1.0.0",
"publisher": "Payara",
"icon": "resources/payara.png",
"repository": {
Expand Down
12 changes: 10 additions & 2 deletions src/main/fish/payara/project/Gradle.ts
Original file line number Diff line number Diff line change
Expand Up @@ -65,13 +65,15 @@ export class Gradle implements Build {
var filename = path.join(buildDir, artifacts[i]);
if (remote) {
if (artifacts[i].endsWith('.war')
|| artifacts[i].endsWith('.jar')) {
|| artifacts[i].endsWith('.jar')
|| artifacts[i].endsWith('.rar')) {
artifact = filename;
break;
}
} else {
if (artifacts[i].endsWith('.war')
|| artifacts[i].endsWith('.jar')
|| artifacts[i].endsWith('.rar')
|| artifacts[i] === this.getBuildReader().getFinalName()) {
artifact = filename;
break;
Expand All @@ -81,7 +83,13 @@ export class Gradle implements Build {
if (artifact !== null) {
callback(artifact);
} else {
vscode.window.showErrorMessage(artifact + ' not found.');
let errorMessage = 'Deployment artifact not found in the target.';
if(remote) {
vscode.window.showErrorMessage(errorMessage
+ ' Make sure the deployment file ends with .jar, .rar, or .war to deploy an application to the remote instance.');
} else {
vscode.window.showErrorMessage(errorMessage);
}
}
}
if (code !== 0) {
Expand Down
12 changes: 10 additions & 2 deletions src/main/fish/payara/project/Maven.ts
Original file line number Diff line number Diff line change
Expand Up @@ -64,13 +64,15 @@ export class Maven implements Build {
var filename = path.join(targetDir, artifacts[i]);
if (remote) {
if (artifacts[i].endsWith('.war')
|| artifacts[i].endsWith('.jar')) {
|| artifacts[i].endsWith('.jar')
|| artifacts[i].endsWith('.rar')) {
artifact = filename;
break;
}
} else {
if (artifacts[i].endsWith('.war')
|| artifacts[i].endsWith('.jar')
|| artifacts[i].endsWith('.rar')
|| artifacts[i] === this.getBuildReader().getFinalName()) {
artifact = filename;
break;
Expand All @@ -80,7 +82,13 @@ export class Maven implements Build {
if (artifact !== null) {
callback(artifact);
} else {
vscode.window.showErrorMessage(artifact + ' not found.');
let errorMessage = 'Deployment artifact not found in the target.';
if(remote) {
vscode.window.showErrorMessage(errorMessage
+ ' Make sure the deployment file ends with .jar, .rar, or .war to deploy an application to the remote instance.');
} else {
vscode.window.showErrorMessage(errorMessage);
}
}
}
if (code !== 0) {
Expand Down

0 comments on commit c6abd00

Please sign in to comment.