Skip to content
This repository was archived by the owner on Dec 12, 2025. It is now read-only.

Conversation

@mircea-cosbuc
Copy link
Member

All Submissions:

  • Have you opened an Issue before filing this PR?
  • Have you signed our CLA?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • Put closes #XXXX in your comment to auto-close the issue that your PR fixes (if such).

Copy link
Collaborator

@lsierant lsierant left a comment

Choose a reason for hiding this comment

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

LGTM, with one question

Copy link
Contributor

@slaskawi slaskawi left a comment

Choose a reason for hiding this comment

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

Adding a LGTM but let's first agree on the dependency versions in https://mongodb.slack.com/archives/CGLP6R2PQ/p1668587574400279

@mircea-cosbuc mircea-cosbuc force-pushed the update-go-version-to-1-19 branch from e84bf35 to 06d1182 Compare November 16, 2022 11:40
@mircea-cosbuc mircea-cosbuc merged commit f178780 into master Nov 17, 2022
@mircea-cosbuc mircea-cosbuc deleted the update-go-version-to-1-19 branch November 17, 2022 09:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants