-
Notifications
You must be signed in to change notification settings - Fork 12
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
👻 fix dockerfile to use release image #87
👻 fix dockerfile to use release image #87
Conversation
The addon is passing non-YAML files in --rules option to the analyzer. For example: when custom targets built with uploaded windup.xml files. This does not seem to hurt anything. The analyzer output mentions that it cannot be parsed as YAML but does not error. Fixing will mainly avoid: confusion; red-herrings while troubleshooting. Signed-off-by: Jeff Ortel <jortel@redhat.com>
upgrade Go 1.20. Replaces gofmt with goimports. Documentation claims it performs the same formatting plus manages/orders imports. Added missing packages in Makefile. Updated the go.mod and ran go mod tidy. closes: konveyor#63 NOT FOR 0.3. --------- Signed-off-by: Jeff Ortel <jortel@redhat.com>
Moving the non system binaries to /usr/local/bin --------- Signed-off-by: Shawn Hurley <shawn@hurley.page>
Signed-off-by: Shawn Hurley <shawn@hurley.page>
Signed-off-by: Shawn Hurley <shawn@hurley.page>
Signed-off-by: Jeff Ortel <jortel@redhat.com>
Latest hub. Fixed in hub addon binding: konveyor/tackle2-hub#629 Signed-off-by: Jeff Ortel <jortel@redhat.com>
…konveyor#80)" This reverts commit 92300c9. Signed-off-by: Jeff Ortel <jortel@redhat.com>
…onveyor#82)" This reverts commit 4ad2bc9. Signed-off-by: Jeff Ortel <jortel@redhat.com>
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
@shawn-hurley we also need to pin the CI to use release-0.3 branch https://github.com/konveyor/analyzer-lsp/blob/a6985630be6b0084784796e743992fbe2d40aef8/.github/workflows/demo-testing.yml#L81 |
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
89a6b3b
to
f95c551
Compare
Signed-off-by: Shawn Hurley <shawn@hurley.page>
f95c551
to
122019a
Compare
updated again to deal with windup shim as release-0.3 |
d94b0a0
to
d0dbb06
Compare
No description provided.