forked from 1Password/typeshare
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Merging upstream changes #9
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…for-empty-target-os update for no target-os
* Only overwrite Codable if it has changed. * Avoid cloning expected output.
* set default log level when not using RUST_LOG * update error output for parsing failures * Use adaptive format for coloring when possible * remove dead_code attribute
Fix duplicate root added to walker.
Only assert if go package is present if generating go types
chore: update changelogs and bump versions for v1.13.1
Use typeshare as command name vs typeshare-cli from package
Update to v1.13.2
* Switching build over to `zigbuild` to support linux arm64 variant * Porting over 1Password#140 to support large number * Adding some default overrides to include base types for stuff such as `HashSet` * Typeshare built for repo * Revert
* Removing docdeploy pipeline for fork repo
## Description * Move mac binary to be manual due to lack of mac os workers * Added `scripts/README.md` ## Test Plan * CI ## Revert Plan * Revert
## Description * We have no need for windows binary at the moment ## Test Plan * CI ## Revert Plan * Revert
## Description * Small fix to force pip3 install * Updated README pre-requisites after recent release ## Test Plan * CI ## Revert Plan * Revert
* For complex enums the default serde serialization is externally tagged, to avoid having to re-write our rust logic to use internal tags via `#[serde(tag = "type", content = "content")]` we are going to add manual support for it in `typescript` * Bumping version to `1.12.1` * Adding a `--dry-run` flag to allow building locally ``` enum MyEnum { blahblah { name: String } blahblah2 { age: u64 } } ``` ``` export type MyEnum = { "type": "blahblah", content: { "name": string }} | { "type": "blahblah2", content: { "age": number }} ``` ``` export type MyEnum = { "blahblah": { "name": string }} | { "blahblah2": { "age": number }} ``` * CI passes * Revert
## Description * Remove --break-packages ## Test Plan * CI ## Revert Plan * Revert
Kakapio
approved these changes
Nov 22, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
New changes