Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Run lint against Go 1.17 to constrain introduction of 1.18 features. #1374

Merged
merged 2 commits into from
Jun 13, 2022

Conversation

nathanhammond
Copy link
Contributor

@nathanhammond nathanhammond commented Jun 13, 2022

@gsoltis had a good minimally-invasive suggestion which gets our linters back working: #1168 (comment)

This PR implements that suggestion. We still build and publish with 1.18, the only real difference is we run lint against 1.17; which will happily break us if we introduce 1.18-specific features.

@vercel
Copy link

vercel bot commented Jun 13, 2022

@nathanhammond is attempting to deploy a commit to the Vercel Team on Vercel.

A member of the Team first needs to authorize it.

@nathanhammond nathanhammond changed the title Run tests against Go 1.17 to constrain introduction of 1.18 features. Run lint against Go 1.17 to constrain introduction of 1.18 features. Jun 13, 2022
@vercel
Copy link

vercel bot commented Jun 13, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Updated
turbo-site ⬜️ Ignored (Inspect) Jun 13, 2022 at 6:12PM (UTC)

@nathanhammond
Copy link
Contributor Author

Reverting this is dependent upon: golang/go#48525

@kodiakhq kodiakhq bot merged commit fc02119 into vercel:main Jun 13, 2022
@nathanhammond nathanhammond deleted the seventeen branch June 14, 2022 01:41
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants