-
Notifications
You must be signed in to change notification settings - Fork 37
chore(deps): bump function-sdk-go to v0.5.0 #107
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
Conversation
Signed-off-by: Jared Watts <[email protected]>
|
Looks like build and lint are failing in the CI environment here, will look into this ASAP edit: done, all checks are green now ✅ |
- .golangci.yml config file migrated with golangci-lint migrate Signed-off-by: Jared Watts <[email protected]>
| @@ -1,206 +1,144 @@ | |||
| run: | |||
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.
this file was migrated with golangci-lint migrate, using the same version of v2.4.0 as is specified in ci.yml
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.
I don't love that this command strips all the commentary from the file. What I've been doing is copying the one from c/c (which I migrated then re-added the commentary to) and updating the import paths.
Not a blocker though.
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.
that's a good suggestion, i didn't like how the comments were stripped either. i'll follow up on it and then merge this PR :)
Signed-off-by: Jared Watts <[email protected]>
Signed-off-by: Jared Watts <[email protected]>
Signed-off-by: Jared Watts <[email protected]>
bobh66
left a comment
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
| @@ -1,206 +1,144 @@ | |||
| run: | |||
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.
I don't love that this command strips all the commentary from the file. What I've been doing is copying the one from c/c (which I migrated then re-added the commentary to) and updating the import paths.
Not a blocker though.
Signed-off-by: Jared Watts <[email protected]>
Description of your changes
This PR simply bumps this template repo to use the latest function-sdk-go v0.5.0: https://github.com/crossplane/function-sdk-go/releases/tag/v0.5.0
The basic commands from the README still work to generate code, run unit tests, test with
crossplane render, and build a function package.Nothing specifically for operation functions has been added in this PR, we are focusing on bumping the SDK dependency. We can add some operations examples in the future as desired.
Fixes #103
I have: