-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Update Dagger and build. #1610
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
Update Dagger and build. #1610
Conversation
SuperQ
commented
Aug 31, 2024
•
edited
Loading
edited
- Update dagger actions to use pinnning.
- Add /dagger to dependabot to keep the Dagger modules up-to-date.
- Update dagger default Go version.
- Update tutorials/whatsup Go version.
- Update actions to use pinning.
65dcd99
to
2a7a491
Compare
Another yak shaving for #1601 |
2a7a491
to
11526a8
Compare
* Update dagger actions to use pinnning. * Add /dagger to dependabot to keep the Dagger modules up-to-date. * Update dagger default Go version. * Update tutorials/whatsup Go version. * Update actions to use pinning. Signed-off-by: SuperQ <[email protected]>
11526a8
to
37027a7
Compare
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 just have one question, but code LGTM
Thanks for working on this :)
@@ -17,15 +17,15 @@ permissions: # added using https://github.com/step-security/secure-repo | |||
jobs: | |||
golangci: | |||
permissions: | |||
contents: read # for actions/checkout to fetch code | |||
contents: read # for actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332 # v4.1.7 |
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.
Was this intentional? This is a comment, so I'm not sure if dependabot will keep this up-to-date as well 🤔
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.
No, this is a sed
skill issue on my part. 😹
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.
Added a commit reverting it then :)
Signed-off-by: Arthur Silva Sens <[email protected]>
* Update Dagger and build * Update dagger actions to use pinnning. * Add /dagger to dependabot to keep the Dagger modules up-to-date. * Update dagger default Go version. * Update tutorials/whatsup Go version. * Update actions to use pinning. Signed-off-by: SuperQ <[email protected]> --------- Signed-off-by: SuperQ <[email protected]> Signed-off-by: Arthur Silva Sens <[email protected]> Co-authored-by: Arthur Silva Sens <[email protected]> Signed-off-by: Eugene <[email protected]>