Skip to content

Conversation

@irees
Copy link
Contributor

@irees irees commented Sep 28, 2025

No description provided.

@github-actions
Copy link

github-actions bot commented Oct 1, 2025

Generated Code Check ✅

All generated code is up to date.

@irees irees marked this pull request as ready for review October 14, 2025 08:03
@irees irees requested a review from Copilot October 14, 2025 08:03
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Implements a transformation filter system for GTFS data processing that allows field-level transformations on entities using various string manipulation functions. This adds capability to apply transforms like uppercase, lowercase, trim, URL escaping, and space-to-underscore replacement.

Key changes:

  • New TransformFilter implementation with string transformation functions
  • Comprehensive test coverage for transformation scenarios
  • Integration with existing filtering infrastructure using SetterFilter patterns

Reviewed Changes

Copilot reviewed 30 out of 30 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
extract/transform_filter.go Core implementation of TransformFilter with transformation functions and entity processing logic
extract/transform_filter_test.go Comprehensive test suite covering all transformation scenarios and edge cases
extract/setterfilter.go Enhanced error handling and wildcard entity matching support
cmds/extract_cmd.go Added exclude-unused-routes functionality for filtering out routes without trips
copier/copier.go Improved error message to include extension type information
doc/cli/*.md Updated auto-generated documentation timestamps

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@irees irees merged commit 7c353de into main Oct 14, 2025
6 checks passed
@irees irees deleted the transform-filter branch October 14, 2025 08:08
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.

2 participants