Skip to content

Add initial RBI for SimpleCov - #362

Open
amomchilov wants to merge 1 commit into
mainfrom
Alex/simplecov
Open

amomchilov wants to merge 1 commit into
mainfrom
Alex/simplecov

Conversation

@amomchilov

Copy link
Copy Markdown
Contributor

Type of Change

  • Add RBI for a new gem

Changes

Comment thread rbi/annotations/simplecov.rbi Outdated
include SimpleCov::FormatterInterface
end

module SimpleCov::MultiFormatter

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.

Where would I add the RBI to declare that SimpleCov::HTMLFormatter conforms to FormatterInterface?

It's defined in a separate simplecov-html gem. I could add a separate simplecov-html.rbi, but then where would it get module SimpleCov::FormatterInterface from?

@amomchilov amomchilov self-assigned this Jan 8, 2026
@amomchilov
amomchilov marked this pull request as ready for review January 8, 2026 17:20
@amomchilov
amomchilov requested a review from a team as a code owner January 8, 2026 17:20
Comment thread rbi/annotations/simplecov.rbi Outdated
Comment thread rbi/annotations/simplecov.rbi Outdated
@amomchilov
amomchilov force-pushed the Alex/simplecov branch 5 times, most recently from b693aa0 to 37ad98f Compare January 9, 2026 01:10
co-authored-by: Alexandre Terrasa <alexandre.terrasa@shopify.com>
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.

2 participants