add section when to return result

[ShahakShama]

Note

Low Risk
Low risk documentation-only change that doesn’t affect runtime behavior. Minor risk of shifting team conventions around error propagation/panics.

Overview
Adds a new “When to return Result” section to style.md, advising to return Result only for errors callers are expected to handle.

Recommends panicking at the source when failure should abort, and handling/logging internally when errors are intentionally ignored, rather than forcing callers to discard results.

^{Written by Cursor Bugbot for commit 0e7c9a6. This will update automatically on new commits. Configure here.}

[reviewable-StarkWare]

This change is 

[ShahakShama]

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

• allow unreachable, and expand when to use panic #13636
• add section when to return result #13635 👈 (View in Graphite)
• error types should have a Error suffix #13627
• add section on expect vs allow #13626
• delete combinators section #13416
• add section on test names #13276
• make short unit test into a recommendation #13275
• improve documentation section #13274
• remove track_caller sentence #13267
• remove sections of re-export #13266
• add logging section #13261 : 1 other dependent PR (#13262 )
• add instruction that TODOs should have an owner #13260
• add section advocating to use Self #13259
• add section on american english vs british english #13257
• remove section about private fields altogether #13166
• unite getter and setter sections, and modify setter section #13165
• reorder API sections (no text change) #13164
• add section saying that code inside macro should be formatted #13163
• add section on future returning pending #13162
• improve ai section #13147
• improve casting section #13146
• make the example in combinators compile #13145
• add comment in return value section to split checking the value and performing action to 2 lines #13144
• remove bad example in foo_or_else section #13141
• improve the starting paragraphs of testing section #13139
• improve macros section #13137
• remove unnecessary misc sections #13131
• reorder sections (move AI and Testing up) #13118
• remove open issues section #13115
• improve external dependencies section #13114
• remove github smell testing section #13113
• create misc section, move stuff into it, and order by importance (no erasure of anything yet) #13112
• change file layout section #13078
• add section on sleep in tests #13073
• add section on dependency injection #13072
• move expect message to a separate section with more explanation #13071
• add directory layout section #12950
• rename allocations and add &Option<T> #12781
• remove laziness  #11591
• rewrite types section #11588
• more comments  #11587
• more comments  #11586
• more comments  #11585
• more comments  #11584
• arithmetic: remove ditto for other mathematical operations. #11848
• gilad/style-guide

This stack of pull requests is managed by Graphite. Learn more about stacking.