Skip to content

Update depencencies#35

Merged
masa213f merged 3 commits intomainfrom
update-20260327
Apr 6, 2026
Merged

Update depencencies#35
masa213f merged 3 commits intomainfrom
update-20260327

Conversation

@masa213f
Copy link
Copy Markdown
Contributor

@masa213f masa213f commented Mar 27, 2026

No description provided.

Comment thread docs/book.toml
@@ -1,6 +1,5 @@
[book]
language = "en"
multilingual = false
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The book.multilingual field had been removed in mdBook v0.5.0.
It seems this field wasn't being used.

https://github.com/rust-lang/mdBook/blob/master/CHANGELOG.md#config-changes
rust-lang/mdBook#2775

}
if paranoid >= 0 {
t.Skipf("Skip test because perf_event_paranoid is %q", paranoid)
t.Skipf("Skip test because perf_event_paranoid is %d", paranoid)
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The test was failing, so I fixed it.

https://github.com/cybozu-go/necoperf/actions/runs/23629613706/job/68825920774?pr=35

# github.com/cybozu-go/necoperf/internal/resource
# [github.com/cybozu-go/necoperf/internal/resource]
Error: internal/resource/perf_test.go:28:53: (*testing.common).Skipf format %q has arg paranoid of wrong type int
FAIL	github.com/cybozu-go/necoperf/internal/resource [build failed]
	github.com/cybozu-go/necoperf/internal/rpc		coverage: 0.0% of statements

@masa213f masa213f requested a review from zeroalphat March 27, 2026 04:10
Comment thread Makefile.versions
@masa213f masa213f force-pushed the update-20260327 branch 5 times, most recently from af95845 to 8afc725 Compare April 3, 2026 07:17
@masa213f masa213f changed the base branch from main to test-perf April 3, 2026 07:18
Base automatically changed from test-perf to main April 6, 2026 07:02
@masa213f masa213f requested a review from zeroalphat April 6, 2026 07:23
Copy link
Copy Markdown
Contributor

@zeroalphat zeroalphat left a comment

Choose a reason for hiding this comment

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

LGTM

@masa213f masa213f merged commit 01e8867 into main Apr 6, 2026
5 checks passed
@masa213f masa213f deleted the update-20260327 branch April 6, 2026 08:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants