Stream: git-wasmtime

Topic: wasmtime / PR #13475 Don't perform excessive growth in ap...


view this post on Zulip Wasmtime GitHub notifications bot (May 24 2026 at 15:47):

alexcrichton requested fitzgen for a review on PR #13475.

view this post on Zulip Wasmtime GitHub notifications bot (May 24 2026 at 15:47):

alexcrichton opened PR #13475 from alexcrichton:api-limit-growth to bytecodealliance:main:

Limit growth to a reasonable number of slots to avoid the growth taking too long. Growth through the API isn't the most optimal at this time.

<!--
Please make sure you include the following information:

Our development process is documented in the Wasmtime book:
https://docs.wasmtime.dev/contributing-development-process.html

Please ensure all communication follows the code of conduct:
https://github.com/bytecodealliance/wasmtime/blob/main/CODE_OF_CONDUCT.md
-->

view this post on Zulip Wasmtime GitHub notifications bot (May 24 2026 at 15:47):

alexcrichton requested wasmtime-fuzz-reviewers for a review on PR #13475.

view this post on Zulip Wasmtime GitHub notifications bot (May 24 2026 at 18:02):

github-actions[bot] added the label fuzzing on PR #13475.

view this post on Zulip Wasmtime GitHub notifications bot (May 24 2026 at 18:03):

github-actions[bot] commented on PR #13475:

Subscribe to Label Action

cc @fitzgen

<details>
This issue or pull request has been labeled: "fuzzing"

Thus the following users have been cc'd because of the following labels:

To subscribe or unsubscribe from this label, edit the <code>.github/subscribe-to-label.json</code> configuration file.

Learn more.
</details>

view this post on Zulip Wasmtime GitHub notifications bot (May 24 2026 at 22:45):

:thumbs_up: angelnereira submitted PR review:

% 10 had modulo bias; int_in_range fixes both the bound and the distribution. The asymmetric caps (100 vs 10) make sense given page cost. LGTM.

view this post on Zulip Wasmtime GitHub notifications bot (May 26 2026 at 17:27):

:thumbs_up: pchickey submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (May 26 2026 at 17:27):

:thumbs_up: fitzgen submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (May 26 2026 at 17:28):

fitzgen added PR #13475 Don't perform excessive growth in api_calls generator to the merge queue.

view this post on Zulip Wasmtime GitHub notifications bot (May 26 2026 at 17:32):

fitzgen commented on PR #13475:

@angelnereira was this an AI comment? Please familiarize yourself with the Bytecode Alliance's AI tool use policy.

Either way, it is strange, verging on anti-social, to review (and especially approve) PRs of projects that you are not a maintainer of. If you had questions or something coming from a place of curiosity, that is one thing, but positioning yourself as a gatekeeper for the project (by stating things with authority and setting the approve or request changes bits) when you are not actually in that position is not welcome.

Thanks.

view this post on Zulip Wasmtime GitHub notifications bot (May 26 2026 at 18:01):

angelnereira commented on PR #13475:

@angelnereira was this an AI comment? Please familiarize yourself with the Bytecode Alliance's AI tool use policy.

Either way, it is strange, verging on anti-social, to review (and especially approve) PRs of projects that you are not a maintainer of. If you had questions or something coming from a place of curiosity, that is one thing, but positioning yourself as a gatekeeper for the project (by stating things with authority and setting the approve or request changes bits) when you are not actually in that position is not welcome.

Thanks.

Thanks for the clarification, and sorry for the noise.

I understand the concern. I did not intend to imply authority over the project or position myself as a gatekeeper. I’ll avoid using formal approve/request-changes reviews on PRs where I’m not a maintainer, and will keep future feedback to questions or clearly non-authoritative comments when appropriate.

I’ll also review the Bytecode Alliance AI tool use policy.

Thanks for pointing this out. :)

view this post on Zulip Wasmtime GitHub notifications bot (May 26 2026 at 20:09):

:check: fitzgen merged PR #13475.

view this post on Zulip Wasmtime GitHub notifications bot (May 26 2026 at 20:09):

fitzgen removed PR #13475 Don't perform excessive growth in api_calls generator from the merge queue.


Last updated: Jun 01 2026 at 09:49 UTC