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

Update Go runtime to 1.20.4 #1002

Merged
merged 3 commits into from
May 3, 2023
Merged

Update Go runtime to 1.20.4 #1002

merged 3 commits into from
May 3, 2023

Conversation

mrodm
Copy link
Contributor

@mrodm mrodm commented May 3, 2023

It includes bugfixes for crypto/tls and net/http packages among others.

Here it can be checked all the changes in this release.

@elasticmachine
Copy link

elasticmachine commented May 3, 2023

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2023-05-03T14:34:42.403+0000

  • Duration: 12 min 41 sec

Test stats 🧪

Test Results
Failed 0
Passed 488
Skipped 0
Total 488

🤖 GitHub comments

Expand to view the GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

CHANGELOG.md Outdated
Comment on lines 13 to 14
* Update Go runtime to 1.20.3. [#987](https://github.com/elastic/package-registry/pull/987)
* Update Go runtime to 1.20.4. [#1002](https://github.com/elastic/package-registry/pull/1002)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I was thinking to just keep one changelog entry for this, like:

* Update Go runtime to 1.20.4. [#1002](https://github.com/elastic/package-registry/pull/1002)

or maybe with the two PR links

* Update Go runtime to 1.20.4. [#987](https://github.com/elastic/package-registry/pull/987) [#1002](https://github.com/elastic/package-registry/pull/1002)

WDYT ?

Copy link
Member

Choose a reason for hiding this comment

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

Yes, keep a single entry for this. Having both links sounds good.

@mrodm mrodm requested a review from a team May 3, 2023 09:53
@mrodm mrodm self-assigned this May 3, 2023
@mrodm mrodm merged commit 9c1cd78 into elastic:main May 3, 2023
@mrodm mrodm deleted the update_go_1_20_4 branch May 3, 2023 15:00
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