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

Release notes v0.52.0 #3805

Merged
merged 21 commits into from
Jun 25, 2024
Merged

Release notes v0.52.0 #3805

merged 21 commits into from
Jun 25, 2024

Conversation

joanlopez
Copy link
Contributor

What?

Release notes for v0.52.0. It is part of the release process defined in #3766.

@joanlopez joanlopez self-assigned this Jun 20, 2024
@joanlopez joanlopez added this to the v0.52.0 milestone Jun 20, 2024
release notes/v0.52.0.md Show resolved Hide resolved
release notes/v0.52.0.md Outdated Show resolved Hide resolved
release notes/v0.52.0.md Outdated Show resolved Hide resolved
Copy link
Contributor

@olegbespalov olegbespalov left a comment

Choose a reason for hiding this comment

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

nit: it will also be good to align the styles across the release notes section where we list issues. Some use a verb with leading capital after the problem, but some do not.

I believe in the past, for most of the time it looks like this:

[#number](link) does ...

release notes/v0.52.0.md Show resolved Hide resolved
release notes/v0.52.0.md Outdated Show resolved Hide resolved
release notes/v0.52.0.md Outdated Show resolved Hide resolved
release notes/v0.52.0.md Outdated Show resolved Hide resolved
release notes/v0.52.0.md Outdated Show resolved Hide resolved
release notes/v0.52.0.md Outdated Show resolved Hide resolved
inancgumus
inancgumus previously approved these changes Jun 20, 2024
@joanlopez
Copy link
Contributor Author

nit: it will also be good to align the styles across the release notes section where we list issues. Some use a verb with leading capital after the problem, but some do not.

I believe in the past, for most of the time it looks like this:

[#number](link) does ...

Gotcha! I can do another pass trying to apply that format. Thanks for the suggestion! 🙇🏻

release notes/v0.52.0.md Outdated Show resolved Hide resolved
release notes/v0.52.0.md Outdated Show resolved Hide resolved
release notes/v0.52.0.md Outdated Show resolved Hide resolved
release notes/v0.52.0.md Outdated Show resolved Hide resolved
release notes/v0.52.0.md Outdated Show resolved Hide resolved
oleiade
oleiade previously approved these changes Jun 21, 2024
release notes/v0.52.0.md Outdated Show resolved Hide resolved
release notes/v0.52.0.md Outdated Show resolved Hide resolved
release notes/v0.52.0.md Outdated Show resolved Hide resolved
release notes/v0.52.0.md Outdated Show resolved Hide resolved
release notes/v0.52.0.md Outdated Show resolved Hide resolved
release notes/v0.52.0.md Outdated Show resolved Hide resolved
release notes/v0.52.0.md Outdated Show resolved Hide resolved
release notes/v0.52.0.md Outdated Show resolved Hide resolved
release notes/v0.52.0.md Outdated Show resolved Hide resolved
Co-authored-by: Heitor Tashiro Sergent <[email protected]>
Co-authored-by: Ivan <[email protected]>
@joanlopez joanlopez dismissed stale reviews from oleiade and inancgumus via 83c7e75 June 25, 2024 08:26
@codecov-commenter
Copy link

codecov-commenter commented Jun 25, 2024

Codecov Report

Attention: Patch coverage is 68.00000% with 120 lines in your changes missing coverage. Please review.

Project coverage is 71.88%. Comparing base (93649df) to head (e037e9e).
Report is 27 commits behind head on master.

Current head e037e9e differs from pull request most recent head c5763c9

Please upload reports for the commit c5763c9 to get more accurate results.

Files Patch % Lines
...odules/k6/experimental/streams/readable_streams.go 27.58% 18 Missing and 3 partials ⚠️
js/modules/k6/experimental/streams/module.go 68.62% 11 Missing and 5 partials ⚠️
js/modules/k6/html/elements.go 0.00% 15 Missing ⚠️
js/modules/k6/html/element.go 15.38% 11 Missing ⚠️
...ntal/streams/readable_stream_default_controller.go 16.66% 3 Missing and 2 partials ⚠️
js/runner.go 76.19% 4 Missing and 1 partial ⚠️
cmd/ui.go 20.00% 3 Missing and 1 partial ⚠️
js/modules/k6/html/elements_gen.go 42.85% 4 Missing ⚠️
js/modules/k6/html/html.go 71.42% 4 Missing ⚠️
js/modules/k6/http/response.go 63.63% 1 Missing and 3 partials ⚠️
... and 17 more
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3805      +/-   ##
==========================================
+ Coverage   70.88%   71.88%   +1.00%     
==========================================
  Files         291      291              
  Lines       21226    21274      +48     
==========================================
+ Hits        15045    15293     +248     
+ Misses       5216     4917     -299     
- Partials      965     1064      +99     
Flag Coverage Δ
ubuntu 71.83% <68.00%> (+1.00%) ⬆️
windows 71.73% <68.00%> (+1.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

codebien
codebien previously approved these changes Jun 25, 2024
mstoykov
mstoykov previously approved these changes Jun 25, 2024
@joanlopez joanlopez mentioned this pull request Jun 25, 2024
30 tasks
@joanlopez joanlopez dismissed stale reviews from mstoykov and codebien via c5763c9 June 25, 2024 10:41
@joanlopez joanlopez merged commit 20f8feb into master Jun 25, 2024
22 checks passed
@joanlopez joanlopez deleted the release-v0.52.0 branch June 25, 2024 10:58
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.

9 participants