Skip to content

Test verbosity fix#1166

Open
apiology wants to merge 12 commits intocastwide:v0.59from
apiology:better_logging_testing
Open

Test verbosity fix#1166
apiology wants to merge 12 commits intocastwide:v0.59from
apiology:better_logging_testing

Conversation

@apiology
Copy link
Contributor

@apiology apiology commented Jan 29, 2026

apiology and others added 6 commits January 13, 2026 05:08
We were using the sig/shims directory for some internally helpful
shims; unfortunately that exported them during gem installs, causing
castwide#1144
* Use each_line instead of text.lines

Avoid allocating additional strings, instead use sliced substrings

* gitignore vendor/cache

* Remove redundant end_with?

Co-authored-by: Jean Boussier <jean.boussier@gmail.com>

* Remove benchamrks

castwide#1054 (comment)

* String#index(offset:) FTW 🚀

castwide#1054 (comment)

Co-authored-by: Jean Boussier <jean.boussier@gmail.com>

* fix rubocop

---------

Co-authored-by: Jean Boussier <jean.boussier@gmail.com>
* Raise InvalidOffsetError for offsets > text

* Linting

* Fix fencepost error

* Additional fencepost test

* Document exception
@apiology apiology changed the title Debug logging fixes Test verbosity fix Jan 29, 2026
@apiology apiology changed the base branch from master to v0.59 January 31, 2026 16:37
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