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

remove all versions < 1.16 and add 1.18 #793

Merged
merged 1 commit into from
Jun 20, 2022
Merged

Conversation

ChannyClaus
Copy link
Contributor

@ChannyClaus ChannyClaus commented Jun 20, 2022

Fixes #792

Summary of Changes
all within .circleci/config.yml:

  1. point the image field to the new image URL.
  2. fix the working directory (the old working directory no longer works against the new image).
  3. drop go versions that are no longer supported + add 1.18

PS: Make sure your PR includes/updates tests! If you need help with this part, just ask!

- image: "circleci/golang:<< parameters.version >>"
working_directory: /go/src/github.com/gorilla/websocket
- image: "cimg/go:<< parameters.version >>"
working_directory: /home/circleci/project/go/src/github.com/gorilla/websocket
Copy link
Contributor Author

Choose a reason for hiding this comment

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

https://circleci.com/docs/2.0/env-vars#using-parameters-and-bash-environment seems like hardcoding is necessary (i.e. can't use $HOME, it seems)

@@ -16,8 +16,8 @@ jobs:
type: string
default: ""
docker:
- image: "circleci/golang:<< parameters.version >>"
working_directory: /go/src/github.com/gorilla/websocket
- image: "cimg/go:<< parameters.version >>"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

https://hub.docker.com/r/cimg/go

This image is designed to supercede the legacy CircleCI Go image, circleci/golang.

@@ -67,4 +67,4 @@ workflows:
- test:
matrix:
parameters:
version: ["latest", "1.17", "1.16", "1.15", "1.14", "1.13", "1.12", "1.11"]
version: ["1.18", "1.17", "1.16"]
Copy link
Contributor Author

@ChannyClaus ChannyClaus Jun 20, 2022

Choose a reason for hiding this comment

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

seems like they no longer publish with :latest tag:

$ docker pull cimg/go
Using default tag: latest
Error response from daemon: manifest for cimg/go:latest not found: manifest unknown: manifest unknown

likely due to ambiguity and confusion of using latest.

@ChannyClaus ChannyClaus marked this pull request as ready for review June 20, 2022 05:29
Copy link

@amustaque97 amustaque97 left a comment

Choose a reason for hiding this comment

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

Thank you for working on this 👏🏻 Changes look good to me 🚢

cc @garyburd

@garyburd garyburd merged commit 27d91a9 into gorilla:master Jun 20, 2022
nono referenced this pull request in cozy/cozy-stack Nov 6, 2023
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [github.com/gorilla/websocket](https://togithub.com/gorilla/websocket)
| require | patch | `v1.5.0` -> `v1.5.1` |

---

### Release Notes

<details>
<summary>gorilla/websocket (github.com/gorilla/websocket)</summary>

###
[`v1.5.1`](https://togithub.com/gorilla/websocket/releases/tag/v1.5.1)

[Compare
Source](https://togithub.com/gorilla/websocket/compare/v1.5.0...v1.5.1)

#### What's Changed

- Add check for Sec-WebSocket-Key header by
[@&#8203;hirasawayuki](https://togithub.com/hirasawayuki) in
[https://github.com/gorilla/websocket/pull/752](https://togithub.com/gorilla/websocket/pull/752)
- Changed the method name UnderlyingConn to NetConn by
[@&#8203;JWSong](https://togithub.com/JWSong) in
[https://github.com/gorilla/websocket/pull/773](https://togithub.com/gorilla/websocket/pull/773)
- remove all versions < 1.16 and add 1.18 by
[@&#8203;ChannyClaus](https://togithub.com/ChannyClaus) in
[https://github.com/gorilla/websocket/pull/793](https://togithub.com/gorilla/websocket/pull/793)
- Check for and report bad protocol in TLSClientConfig.NextProtos by
[@&#8203;ChannyClaus](https://togithub.com/ChannyClaus) in
[https://github.com/gorilla/websocket/pull/788](https://togithub.com/gorilla/websocket/pull/788)
- check err before GotConn for trace by
[@&#8203;junnplus](https://togithub.com/junnplus) in
[https://github.com/gorilla/websocket/pull/798](https://togithub.com/gorilla/websocket/pull/798)
- Update README.md by
[@&#8203;coreydaley](https://togithub.com/coreydaley) in
[https://github.com/gorilla/websocket/pull/839](https://togithub.com/gorilla/websocket/pull/839)
- Correct way to save memory using write buffer pool and freeing
net.http default buffers by [@&#8203;FMLS](https://togithub.com/FMLS) in
[https://github.com/gorilla/websocket/pull/761](https://togithub.com/gorilla/websocket/pull/761)
- Update go version & add verification/testing tools by
[@&#8203;coreydaley](https://togithub.com/coreydaley) in
[https://github.com/gorilla/websocket/pull/840](https://togithub.com/gorilla/websocket/pull/840)
- update golang.org/x/net by
[@&#8203;coreydaley](https://togithub.com/coreydaley) in
[https://github.com/gorilla/websocket/pull/856](https://togithub.com/gorilla/websocket/pull/856)
- update GitHub workflows by
[@&#8203;coreydaley](https://togithub.com/coreydaley) in
[https://github.com/gorilla/websocket/pull/857](https://togithub.com/gorilla/websocket/pull/857)

#### New Contributors

- [@&#8203;hirasawayuki](https://togithub.com/hirasawayuki) made their
first contribution in
[https://github.com/gorilla/websocket/pull/752](https://togithub.com/gorilla/websocket/pull/752)
- [@&#8203;JWSong](https://togithub.com/JWSong) made their first
contribution in
[https://github.com/gorilla/websocket/pull/773](https://togithub.com/gorilla/websocket/pull/773)
- [@&#8203;ChannyClaus](https://togithub.com/ChannyClaus) made their
first contribution in
[https://github.com/gorilla/websocket/pull/793](https://togithub.com/gorilla/websocket/pull/793)
- [@&#8203;junnplus](https://togithub.com/junnplus) made their first
contribution in
[https://github.com/gorilla/websocket/pull/798](https://togithub.com/gorilla/websocket/pull/798)
- [@&#8203;coreydaley](https://togithub.com/coreydaley) made their first
contribution in
[https://github.com/gorilla/websocket/pull/839](https://togithub.com/gorilla/websocket/pull/839)
- [@&#8203;FMLS](https://togithub.com/FMLS) made their first
contribution in
[https://github.com/gorilla/websocket/pull/761](https://togithub.com/gorilla/websocket/pull/761)

**Full Changelog**:
gorilla/websocket@v1.5.0...v1.5.1

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "before 6am on Monday" in timezone
Europe/Paris, Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/cozy/cozy-stack).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4zMS41IiwidXBkYXRlZEluVmVyIjoiMzcuMzEuNSIsInRhcmdldEJyYW5jaCI6Im1hc3RlciJ9-->
algitbot pushed a commit to alpinelinux/build-server-status that referenced this pull request May 5, 2024
This MR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [github.com/gorilla/websocket](https://github.com/gorilla/websocket) | require | patch | `v1.5.0` -> `v1.5.1` |

---

### Release Notes

<details>
<summary>gorilla/websocket (github.com/gorilla/websocket)</summary>

### [`v1.5.1`](https://github.com/gorilla/websocket/releases/tag/v1.5.1)

[Compare Source](gorilla/websocket@v1.5.0...v1.5.1)

#### What's Changed

-   Add check for Sec-WebSocket-Key header by [@&#8203;hirasawayuki](https://github.com/hirasawayuki) in gorilla/websocket#752
-   Changed the method name UnderlyingConn to NetConn by [@&#8203;JWSong](https://github.com/JWSong) in gorilla/websocket#773
-   remove all versions < 1.16 and add 1.18 by [@&#8203;ChannyClaus](https://github.com/ChannyClaus) in gorilla/websocket#793
-   Check for and report bad protocol in TLSClientConfig.NextProtos by [@&#8203;ChannyClaus](https://github.com/ChannyClaus) in gorilla/websocket#788
-   check err before GotConn for trace by [@&#8203;junnplus](https://github.com/junnplus) in gorilla/websocket#798
-   Update README.md by [@&#8203;coreydaley](https://github.com/coreydaley) in gorilla/websocket#839
-   Correct way to save memory using write buffer pool and freeing net.http default buffers by [@&#8203;FMLS](https://github.com/FMLS) in gorilla/websocket#761
-   Update go version & add verification/testing tools by [@&#8203;coreydaley](https://github.com/coreydaley) in gorilla/websocket#840
-   update golang.org/x/net by [@&#8203;coreydaley](https://github.com/coreydaley) in gorilla/websocket#856
-   update GitHub workflows by [@&#8203;coreydaley](https://github.com/coreydaley) in gorilla/websocket#857

#### New Contributors

-   [@&#8203;hirasawayuki](https://github.com/hirasawayuki) made their first contribution in gorilla/websocket#752
-   [@&#8203;JWSong](https://github.com/JWSong) made their first contribution in gorilla/websocket#773
-   [@&#8203;ChannyClaus](https://github.com/ChannyClaus) made their first contribution in gorilla/websocket#793
-   [@&#8203;junnplus](https://github.com/junnplus) made their first contribution in gorilla/websocket#798
-   [@&#8203;coreydaley](https://github.com/coreydaley) made their first contribution in gorilla/websocket#839
-   [@&#8203;FMLS](https://github.com/FMLS) made their first contribution in gorilla/websocket#761

**Full Changelog**: gorilla/websocket@v1.5.0...v1.5.1

</details>

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this MR, check this box

---

&nbsp;
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4yODYuMSIsInVwZGF0ZWRJblZlciI6IjM3LjI4Ni4xIiwidGFyZ2V0QnJhbmNoIjoibWFzdGVyIiwibGFiZWxzIjpbXX0=-->

See merge request alpine/infra/build-server-status!9
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.

Update Go versions used in CI
3 participants