Skip to content

delete combinators section#13416

Open
ShahakShama wants to merge 1 commit intoshahak/style-guide-test-namesfrom
shahak/delete_combinators_section
Open

delete combinators section#13416
ShahakShama wants to merge 1 commit intoshahak/style-guide-test-namesfrom
shahak/delete_combinators_section

Conversation

@ShahakShama
Copy link
Copy Markdown
Collaborator

@ShahakShama ShahakShama commented Mar 23, 2026

Note

Low Risk
Documentation-only change with no impact on runtime behavior or APIs.

Overview
Removes the Combinators guidance section from style.md, including the recommendation to avoid side effects in map/filter/zip and the accompanying BAD/GOOD example.

Written by Cursor Bugbot for commit 694e16c. This will update automatically on new commits. Configure here.

@ShahakShama ShahakShama requested a review from Stavbe March 23, 2026 12:24
@reviewable-StarkWare
Copy link
Copy Markdown

This change is Reviewable

@ShahakShama ShahakShama force-pushed the shahak/style-guide-test-names branch from 01a819e to 8e233b9 Compare March 26, 2026 07:20
@ShahakShama ShahakShama force-pushed the shahak/delete_combinators_section branch from 183e706 to 6ceac1a Compare March 26, 2026 07:20
@ShahakShama ShahakShama force-pushed the shahak/style-guide-test-names branch from 8e233b9 to beca173 Compare March 26, 2026 07:27
@ShahakShama ShahakShama force-pushed the shahak/delete_combinators_section branch from 6ceac1a to 83b0fa0 Compare March 26, 2026 07:27
@ShahakShama ShahakShama force-pushed the shahak/style-guide-test-names branch from beca173 to bb84c16 Compare March 26, 2026 07:44
@ShahakShama ShahakShama force-pushed the shahak/delete_combinators_section branch 2 times, most recently from 1c6d5c4 to 694e16c Compare March 26, 2026 09:24
@ShahakShama ShahakShama force-pushed the shahak/style-guide-test-names branch from bb84c16 to 0448af8 Compare March 26, 2026 09:24
Copy link
Copy Markdown
Contributor

@Stavbe Stavbe left a comment

Choose a reason for hiding this comment

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

@Stavbe reviewed 1 file and all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on ShahakShama).

@ShahakShama ShahakShama force-pushed the shahak/delete_combinators_section branch from 694e16c to 7ef43a2 Compare April 9, 2026 10:28
@ShahakShama ShahakShama force-pushed the shahak/style-guide-test-names branch 2 times, most recently from 33c2542 to 8780138 Compare April 9, 2026 11:50
@ShahakShama ShahakShama force-pushed the shahak/delete_combinators_section branch 2 times, most recently from 7d5213a to d9860c9 Compare April 9, 2026 12:12
@ShahakShama ShahakShama force-pushed the shahak/style-guide-test-names branch from 32661c4 to 6c01bd8 Compare April 9, 2026 12:15
@ShahakShama ShahakShama force-pushed the shahak/delete_combinators_section branch from d9860c9 to 33332ed Compare April 9, 2026 12:15
@ShahakShama ShahakShama force-pushed the shahak/style-guide-test-names branch from 6c01bd8 to 36c715b Compare April 9, 2026 12:37
@ShahakShama ShahakShama force-pushed the shahak/delete_combinators_section branch from 33332ed to 2eb38fc Compare April 9, 2026 12:37
@ShahakShama ShahakShama force-pushed the shahak/delete_combinators_section branch from 2eb38fc to 7a291b1 Compare April 9, 2026 15:05
@ShahakShama ShahakShama force-pushed the shahak/style-guide-test-names branch from 36c715b to 3afc1cd Compare April 9, 2026 15:05
@ShahakShama ShahakShama force-pushed the shahak/delete_combinators_section branch from 7a291b1 to 78f8f57 Compare April 13, 2026 10:51
@ShahakShama ShahakShama force-pushed the shahak/style-guide-test-names branch from 0b8b63c to 8f19659 Compare April 15, 2026 05:53
@ShahakShama ShahakShama force-pushed the shahak/delete_combinators_section branch from 78f8f57 to 23275c8 Compare April 15, 2026 05:53
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