8000 chore: remove group by direction feature flag by BrandonTR · Pull Request #957 · mbta/mobile_app · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

chore: remove group by direction feature flag #957

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 12 commits into from
May 7, 2025
Merged

Conversation

BrandonTR
Copy link
Contributor
@BrandonTR BrandonTR commented May 6, 2025

Summary

Ticket: Group by direction | Remove feature flag

Removes the group by direction feature flag, this includes Melody's StopDetailsViewTest modifications to reduce merge conflicts!

Testing

Removed and tweaked tests that relied on feature flag.

@BrandonTR BrandonTR marked this pull request as ready for review May 6, 2025 19:16
@BrandonTR BrandonTR requested a review from a team as a code owner May 6, 2025 19:16
@BrandonTR BrandonTR requested a review from boringcactus May 6, 2025 19:16
@@ -50,7 +50,6 @@
{
"parallelizable" : true,
"skippedTests" : [
"EndToEndOpenStopDetailsTest\/testOpenStopDetails()",
Copy link
Member

Choose a reason for hiding this comment

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

This change should also be made in iosAppRetries.xctestplan.

@BrandonTR
Copy link
Contributor Author
Screenshot 2025-05-07 at 11 39 40 AM

Showing tests passing locally for posterity

@BrandonTR BrandonTR added this pull request to the merge queue May 7, 2025
Merged via the queue into main with commit 2872c2a May 7, 2025
10 checks passed
@BrandonTR BrandonTR deleted the br-remove-feature-flag branch May 7, 2025 17:16
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
0