Skip to content

Introduce Options object#954

Merged
vinistock merged 1 commit into
mainfrom
Alex/Options-object
Jun 19, 2026
Merged

Introduce Options object#954
vinistock merged 1 commit into
mainfrom
Alex/Options-object

Conversation

@amomchilov

@amomchilov amomchilov commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

This PR introduces a more complex Options object to allow us to have different sets of configuration. For example, max line length does not apply to the new line preserving format (which may need to create very long lines to preserve backtraces).

The idea is to have a base object that gets specialized for the different cases.

@amomchilov

amomchilov commented Jun 16, 2026

Copy link
Copy Markdown
Contributor Author

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

@amomchilov amomchilov force-pushed the Alex/Options-object branch 3 times, most recently from d5692f6 to 7654edc Compare June 17, 2026 19:01
@vinistock vinistock force-pushed the Alex/Options-object branch from 7654edc to 42fbc58 Compare June 19, 2026 15:00
@vinistock vinistock marked this pull request as ready for review June 19, 2026 15:02
@vinistock vinistock requested a review from a team as a code owner June 19, 2026 15:02
@vinistock vinistock force-pushed the Alex/Options-object branch from 42fbc58 to 9d80524 Compare June 19, 2026 15:12
Comment thread lib/spoom/sorbet/translate/rbs_comments_to_sorbet_sigs/options.rb Outdated
Comment thread lib/spoom/sorbet/translate/rbs_comments_to_sorbet_sigs.rb Outdated
Co-authored-by: Vinicius Stock <vinistock@users.noreply.github.com>
@vinistock vinistock force-pushed the Alex/Options-object branch from 9d80524 to 48c4e23 Compare June 19, 2026 15:22
@vinistock vinistock merged commit d02f7a3 into main Jun 19, 2026
11 checks passed
@vinistock vinistock deleted the Alex/Options-object branch June 19, 2026 17:35
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