Skip to content

Fix property ordering transforms for google payloads#189

Merged
Ken Jiang (knjiang) merged 1 commit intomainfrom
04-10-fix_property_ordering
Apr 10, 2026
Merged

Fix property ordering transforms for google payloads#189
Ken Jiang (knjiang) merged 1 commit intomainfrom
04-10-fix_property_ordering

Conversation

@knjiang
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Contributor Author

Ken Jiang (knjiang) commented Apr 10, 2026

This stack of pull requests is managed by Graphite. Learn more about stacking.

@knjiang Ken Jiang (knjiang) marked this pull request as ready for review April 10, 2026 16:59
@knjiang Ken Jiang (knjiang) changed the title fix property ordering Fix property ordering transforms for google payloads Apr 10, 2026
fn normalize_for_target(
&mut self,
target_provider: ProviderFormat,
strip_google_only_fields: bool,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

why do we need this? can we use target_provider instead?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

ya i think we can just use target_provider.

@knjiang Ken Jiang (knjiang) force-pushed the 04-10-fix_property_ordering branch from 60e0468 to 0ad030e Compare April 10, 2026 21:09
@knjiang Ken Jiang (knjiang) force-pushed the 04-10-fix_property_ordering branch from 0ad030e to 63fd234 Compare April 10, 2026 21:30
@knjiang Ken Jiang (knjiang) merged commit ad02ee4 into main Apr 10, 2026
11 checks passed
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