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

Go: Bump github.com/pborman/uuid from 1.2.0 to 1.2.1 #5901

Merged

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Jun 29, 2022

Bumps github.com/pborman/uuid from 1.2.0 to 1.2.1.

Release notes

Sourced from github.com/pborman/uuid's releases.

Fix golint warning

This release contain fixes by KosToZyB for golint warnings.

Commits

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

@dependabot dependabot bot added area/dependency Issues or PRs related to dependency changes. This label is also used on all Dependabot PRs kind/task Issue is actionable task ok-to-test Indicates a non-member PR verified by an org member that is safe to test. labels Jun 29, 2022
@openshift-ci openshift-ci bot requested review from cdrage and feloy June 29, 2022 15:17
@openshift-ci
Copy link

openshift-ci bot commented Jun 29, 2022

Hi @dependabot[bot]. Thanks for your PR.

I'm waiting for a redhat-developer member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@netlify
Copy link

netlify bot commented Jun 29, 2022

Deploy Preview for odo-docusaurus-preview ready!

Name Link
🔨 Latest commit 58a5940
🔍 Latest deploy log https://app.netlify.com/sites/odo-docusaurus-preview/deploys/62c5b268aadb670008151318
😎 Deploy Preview https://deploy-preview-5901--odo-docusaurus-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@odo-robot
Copy link

odo-robot bot commented Jun 29, 2022

Unit Tests on commit b705af9 finished successfully.
View logs: TXT HTML

@odo-robot
Copy link

odo-robot bot commented Jun 29, 2022

Kubernetes Tests on commit b705af9 finished successfully.
View logs: TXT HTML

@odo-robot
Copy link

odo-robot bot commented Jun 29, 2022

Windows Tests (OCP) on commit b705af9 finished successfully.
View logs: TXT HTML

@odo-robot
Copy link

odo-robot bot commented Jun 29, 2022

OpenShift Tests on commit b705af9 finished successfully.
View logs: TXT HTML

@odo-robot
Copy link

odo-robot bot commented Jun 29, 2022

Validate Tests on commit b705af9 finished successfully.
View logs: TXT HTML

@rm3l
Copy link
Member

rm3l commented Jul 1, 2022

/hold

Waiting until #5809 is merged (so we don't introduce additional work), and then we'll see how we could update Dependabot PRs to work with the generate-mod.sh script introduced in #5809.

@openshift-ci openshift-ci bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. Required by Prow. label Jul 1, 2022
Bumps [github.com/pborman/uuid](https://github.com/pborman/uuid) from 1.2.0 to 1.2.1.
- [Release notes](https://github.com/pborman/uuid/releases)
- [Commits](pborman/uuid@v1.2...v1.2.1)

---
updated-dependencies:
- dependency-name: github.com/pborman/uuid
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot force-pushed the dependabot/go_modules/github.com/pborman/uuid-1.2.1 branch from 74ce7cd to bd483db Compare July 6, 2022 10:33
rm3l added 2 commits July 6, 2022 17:50
This also sets it as a direct dependency of odo,
as we are using it.
@sonarcloud
Copy link

sonarcloud bot commented Jul 6, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@openshift-ci
Copy link

openshift-ci bot commented Jul 6, 2022

@dependabot: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/unit bd483db link true /test unit

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@rm3l
Copy link
Member

rm3l commented Jul 7, 2022

Canceling the hold now that #5809 has been merged.

/hold cancel

@feloy We saw that together already, but could you please review this PR by taking a look at the last 2 commits that modify the scripts/generate-mod.sh file accordingly.

@openshift-ci openshift-ci bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. Required by Prow. label Jul 7, 2022
@feloy
Copy link
Contributor

feloy commented Jul 12, 2022

/lgtm
/approve

/override ci/prow/unit-and-validate-test
/override ci/prow/v4.10-integration-e2e

@openshift-ci
Copy link

openshift-ci bot commented Jul 12, 2022

@feloy: Overrode contexts on behalf of feloy: ci/prow/unit-and-validate-test, ci/prow/v4.10-integration-e2e

In response to this:

/lgtm
/approve

/override ci/prow/unit-and-validate-test
/override ci/prow/v4.10-integration-e2e

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. Required by Prow. label Jul 12, 2022
@openshift-ci
Copy link

openshift-ci bot commented Jul 12, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: feloy

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. Required by Prow. label Jul 12, 2022
@openshift-ci openshift-ci bot merged commit 46e47a5 into main Jul 12, 2022
@dependabot dependabot bot deleted the dependabot/go_modules/github.com/pborman/uuid-1.2.1 branch July 12, 2022 07:23
cdrage pushed a commit to cdrage/odo that referenced this pull request Aug 31, 2022
…r#5901)

* Go: Bump github.com/pborman/uuid from 1.2.0 to 1.2.1

Bumps [github.com/pborman/uuid](https://github.com/pborman/uuid) from 1.2.0 to 1.2.1.
- [Release notes](https://github.com/pborman/uuid/releases)
- [Commits](pborman/uuid@v1.2...v1.2.1)

---
updated-dependencies:
- dependency-name: github.com/pborman/uuid
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

* Add github.com/pborman/uuid to generate-mod.sh script

This also sets it as a direct dependency of odo,
as we are using it.

* Pin github.com/openshift/oc and k8s.io/utils as direct dependencies in generate-mod.sh script

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Armel Soro <asoro@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. Required by Prow. area/dependency Issues or PRs related to dependency changes. This label is also used on all Dependabot PRs kind/task Issue is actionable task lgtm Indicates that a PR is ready to be merged. Required by Prow. ok-to-test Indicates a non-member PR verified by an org member that is safe to test.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants