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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(deps): bump go to v1.16 #699

Closed
wants to merge 2 commits into from
Closed

chore(deps): bump go to v1.16 #699

wants to merge 2 commits into from

Conversation

chenrui333
Copy link
Contributor

@chenrui333 chenrui333 commented Feb 20, 2021

馃憢 go 1.16 use go-modules build by default, since this is CLI app, there is no reason to do multiple go versions support. Hence, I have dropped the old go versions.

@CLAassistant
Copy link

CLAassistant commented Feb 20, 2021

CLA assistant check
All committers have signed the CLA.

@zlesnr
Copy link
Contributor

zlesnr commented Feb 23, 2021

Hello, thank you for the PR. That sounds right to me also. It was probably a cary-over from another project, but I see your point. I'll upgrade my local system and run the tests.

@zlesnr
Copy link
Contributor

zlesnr commented Feb 23, 2021

It doesn't look like brew has 1.16 yet so I didn't test locally. I think we want to hold this open for a week or so for reasons I can't share yet, but we're on board with this change.

@ctrombley ctrombley changed the base branch from master to main April 30, 2021 18:47
@jpvajda jpvajda added this to Ready for review in NR1 Developer Toolkit Community May 24, 2021
@chenrui333
Copy link
Contributor Author

revisit this PR today

Signed-off-by: Rui Chen <rui@chenrui.dev>
@chenrui333
Copy link
Contributor Author

chenrui333 commented Jun 17, 2021

relates to the new homebrew version bump

NR1 Developer Toolkit Community automation moved this from Ready for review to Review completed Jul 13, 2021
@ctrombley
Copy link
Contributor

ctrombley commented Jul 14, 2021

Thanks @chenrui333 , we will continue this effort in #972 .

@ctrombley ctrombley closed this Jul 14, 2021
NR1 Developer Toolkit Community automation moved this from Review completed to Done Jul 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

4 participants