Skip to content
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

Removing unused reduced response from various Attribute methods #2630

Merged
merged 4 commits into from
Jan 10, 2025

Conversation

mineralntl
Copy link
Collaborator

Picking up where #1458 left off due to staleness. This seeks to remove unused reducedResponse parameters from some of the Attribute uses across the codebase.

@mineralntl mineralntl force-pushed the task/removeUnusedReducedResponse branch from 9c5d985 to 3f89c8a Compare November 18, 2024 15:12
apmoriarty
apmoriarty previously approved these changes Nov 18, 2024
@mineralntl mineralntl force-pushed the task/removeUnusedReducedResponse branch 4 times, most recently from c118409 to 1848ab8 Compare November 25, 2024 16:33
@mineralntl mineralntl force-pushed the task/removeUnusedReducedResponse branch from 1848ab8 to 89b501a Compare November 27, 2024 15:45
@mineralntl mineralntl force-pushed the task/removeUnusedReducedResponse branch 6 times, most recently from 70dd54b to 4dc12aa Compare December 11, 2024 14:59
@mineralntl mineralntl force-pushed the task/removeUnusedReducedResponse branch 2 times, most recently from 37a474a to 449c53e Compare December 16, 2024 15:15
@mineralntl mineralntl force-pushed the task/removeUnusedReducedResponse branch from 449c53e to 5c59228 Compare December 18, 2024 14:48
@mineralntl mineralntl force-pushed the task/removeUnusedReducedResponse branch from 5c59228 to 81ec564 Compare December 31, 2024 00:45
@mineralntl mineralntl force-pushed the task/removeUnusedReducedResponse branch from 81ec564 to b03b798 Compare January 8, 2025 18:59
@mineralntl mineralntl force-pushed the task/removeUnusedReducedResponse branch from b03b798 to 79c3414 Compare January 10, 2025 14:54
@hlgp hlgp merged commit a6d488b into integration Jan 10, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants