build: refactor Rakefile by splitting tasks into separate files #844
+75
−73
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Refactored the monolithic 79-line Rakefile into a clean 15-line loader that dynamically imports task definitions from separate files in the
tasks/directory.Changes
New Structure
tasks/directory with 5 focused rake files:tasks/test.rake- test execution taskstasks/rubocop.rake- RuboCop linting taskstasks/yard.rake- YARD documentation taskstasks/gem_tasks.rake- gem building and release taskstasks/test_gem.rake- gem sanity check taskMain Rakefile
.rakefiles fromtasks/directoryPlatform Logic
yard.rakewhere it belongsMaintained Functionality
All original functionality is preserved:
test,rubocop,yard,yardstick:coverage,buildyardstickremains commented out of defaults (too many warnings)Benefits
Testing
Summary
Remove redundant regexp escape in
BRANCH_LINE_REGEXPflagged by new RuboCop cop Style/RedundantRegexpEscape.Details
Before:
(?<detached_ref>[^\)]+)After:
(?<detached_ref>[^)]+)This is a purely stylistic cleanup with no behavior change.
Rationale
RuboCop now detects unnecessary escaping of
)inside a character class. Removing it reduces noise and keeps style consistent.Verification
bundle exec rubocopreports zero offenses after the change.rake testreports zero failures.