Skip to content

impl(internal/command): delete maybeGetLanguageRepo, maybeLoadStateAndConfig, and execute#201

Closed
julieqiu wants to merge 2 commits intopr6from
pr7
Closed

impl(internal/command): delete maybeGetLanguageRepo, maybeLoadStateAndConfig, and execute#201
julieqiu wants to merge 2 commits intopr6from
pr7

Conversation

@julieqiu
Copy link
Copy Markdown
Member

@julieqiu julieqiu commented Jun 4, 2025

No description provided.

@julieqiu julieqiu requested a review from a team June 4, 2025 02:25
@julieqiu julieqiu changed the base branch from main to pr6 June 4, 2025 02:26
@julieqiu julieqiu mentioned this pull request Jun 4, 2025
@julieqiu julieqiu force-pushed the pr6 branch 3 times, most recently from 28cf8c2 to 8f97338 Compare June 4, 2025 04:51
@julieqiu julieqiu closed this Jun 4, 2025
@julieqiu julieqiu deleted the pr7 branch June 11, 2025 11:41
julieqiu pushed a commit that referenced this pull request Jul 30, 2025
Some languages (Rust included) will need attributes or some other kind
of annotation preceeding each field.

In Rust's case, we need to add some attributes to control how `serde`
serializes and deserializes 64-bit integers and bytes.

This PR also refactors some tests in `types/` to show that the
attributes work as expected, and regenerates the golden files with the
new attributes.
julieqiu pushed a commit that referenced this pull request Jul 31, 2025
Some languages (Rust included) will need attributes or some other kind
of annotation preceeding each field.

In Rust's case, we need to add some attributes to control how `serde`
serializes and deserializes 64-bit integers and bytes.

This PR also refactors some tests in `types/` to show that the
attributes work as expected, and regenerates the golden files with the
new attributes.
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.

1 participant