search

fabric8-analytics / fabric8-analytics-lsp-server

Apache License 2.0
9 stars 33 forks source link

fix(semantic-release): Update semantic-release to latest #109

Closed arajkumar closed 4 years ago

arajkumar commented 5 years ago
centos-ci commented 5 years ago

Can one of the admins verify this patch?

arajkumar commented 5 years ago

[test]

arajkumar commented 5 years ago

@invincibleJai

I did used my personal npmjs account and forked github repo to test this change, something like below,

diff --git b/package.json a/package.json
index 6ee379b..179cbf2 100644
--- b/package.json
+++ a/package.json
@@ -1,5 +1,5 @@
 {
-  "name": "fabric8-analytics-lsp-server",
+  "name": "ararunrajkumar-lsp-server-demo",
   "description": "LSP Server for Dependency Analytics",
   "version": "0.0.0-development",
   "author": "Pavel Odvody",
@@ -27,7 +27,7 @@
   ],
   "repository": {
     "type": "git",
-    "url": "https://github.com/fabric8-analytics/fabric8-analytics-lsp-server.git"
+    "url": "https://github.com/arajkumar/fabric8-analytics-lsp-server.git"
   },
   "dependencies": {
     "request": "^2.79.0",
invincibleJai commented 5 years ago

@arajkumar looks like CI failed for this change, let me retrigger it https://ci.centos.org/job/devtools-fabric8-analytics-lsp-server/88/

invincibleJai commented 5 years ago

[test]

arajkumar commented 5 years ago

@arajkumar looks like CI failed for this change, let me retrigger it https://ci.centos.org/job/devtools-fabric8-analytics-lsp-server/88/

There is a typo in the script. I will fix it ASAP. Thanks!

arajkumar commented 5 years ago

[test]

arajkumar commented 5 years ago

[test]

arajkumar commented 5 years ago

@invincibleJai Looks like epel-release package is not required for this component, similar like vscode extension. Now the build is passing in CI.

arajkumar commented 4 years ago

[test]

dvandra commented 4 years ago

@arajkumar can we go ahead and merge this PR??