Skip to content

Conversation

@alfrunes
Copy link
Contributor

  • github.com/gorilla/websocket v1.5.0 -> v1.5.1
  • github.com/mendersoftware/go-lib-micro v0.0.0-20230808081028-48c0b853fd99 -> v0.0.0-20231031082957-e2bce6ff3edd
  • github.com/urfave/cli/v2 v2.0.0-00010101000000-000000000000 -> v2.25.7

@kjaskiewiczz
Copy link
Contributor

I think we are lacking some checks in this repository - gitlab pipeline failed, nobody approved and I can still merge the PR.

@olehermanse
Copy link
Contributor

@kjaskiewiczz Thanks, I've added some basic branch protection.

Copy link
Contributor

@kjaskiewiczz kjaskiewiczz left a comment

Choose a reason for hiding this comment

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

looks good, but the license checker is not happy

Copy link
Contributor

@olehermanse olehermanse left a comment

Choose a reason for hiding this comment

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

LGTM, except for CI.

@alfrunes
Copy link
Contributor Author

I dropped the commit removing the vendor folder and LIC_FILES_CHKSUM.sha256 since this is a device component and requires compatibility with Yocto (as pointed out here).

* github.com/gorilla/websocket v1.5.0 -> v1.5.1
* github.com/mendersoftware/go-lib-micro v0.0.0-20230808081028-48c0b853fd99 -> v0.0.0-20231031082957-e2bce6ff3edd
* github.com/urfave/cli/v2 v2.0.0-00010101000000-000000000000 -> v2.25.7

Signed-off-by: Alf-Rune Siqveland <alf.rune@northern.tech>
@alfrunes alfrunes requested a review from olehermanse November 15, 2023 11:57
@olehermanse olehermanse changed the title Dependency upgrade and remove vendor directory Dependency upgrades Nov 16, 2023
@alfrunes alfrunes merged commit 8100448 into NorthernTechHQ:main Nov 16, 2023
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.

3 participants