Skip to content
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

Closed

Conversation

shawn-hurley
Copy link
Contributor

No description provided.

jortel and others added 9 commits January 19, 2024 10:17
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 <[email protected]>
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 <[email protected]>
Moving the non system binaries to /usr/local/bin

---------

Signed-off-by: Shawn Hurley <[email protected]>
Latest hub.
Fixed in hub addon binding:
konveyor/tackle2-hub#629

Signed-off-by: Jeff Ortel <[email protected]>
Copy link
Contributor

@jortel jortel left a comment

Choose a reason for hiding this comment

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

LGTM

@pranavgaikwad
Copy link

Copy link
Member

@aufi aufi left a comment

Choose a reason for hiding this comment

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

LGTM

@shawn-hurley shawn-hurley force-pushed the feature/fix-dockerfile branch 3 times, most recently from 89a6b3b to f95c551 Compare April 30, 2024 13:52
@shawn-hurley
Copy link
Contributor Author

updated again to deal with windup shim as release-0.3

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.

5 participants