8000 feat(si-pkg,object-tree): add optional kind key to SiPkg by zacharyhamm · Pull Request #2709 · systeminit/si · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

feat(si-pkg,object-tree): add optional kind key to SiPkg #2709

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 1 commit into from
Aug 28, 2023

Conversation

zacharyhamm
Copy link
Contributor

Adds a new key to the root node of an SiPkg that expresses the "kind" of a package (currently SiPkgKind::Module and SiPkgKind::WorkspaceBackup). The "kind" key is optional, and packages without (like the existing modules), default to SiPkgKind::Module. To support this, a new reader function was added to object-tree to support reading a key that may or may not be in the expected position (read_key_line_value_opt).

@zacharyhamm zacharyhamm force-pushed the zack/add-kind-to-pkg-metadata-backwards-compat branch 2 times, most recently from 29b9139 to c4f0a98 Compare August 28, 2023 17:54
@zacharyhamm zacharyhamm marked this pull request as ready for review August 28, 2023 18:15
@zacharyhamm zacharyhamm force-pushed the zack/add-kind-to-pkg-metadata-backwards-compat branch from c4f0a98 to 9e07294 Compare August 28, 2023 18:31
@zacharyhamm
Copy link
Contributor Author

bors try

si-bors-ng bot added a commit that referenced this pull request Aug 28, 2023
Adds a new key to the root node of an SiPkg that expresses the "kind" of
a package (currently `SiPkgKind::Module` and `SiPkgKind::WorkspaceBackup`).
The "kind" key is optional, and packages without (like the existing
modules), default to `SiPkgKind::Module`. To support this, a new reader
function was added to `object-tree` to support reading a key that may or
may not be in the expected position (`read_key_line_value_opt`).
@zacharyhamm zacharyhamm force-pushed the zack/add-kind-to-pkg-metadata-backwards-compat branch from 9e07294 to ffa08b3 Compare August 28, 2023 18:53
@si-bors-ng
Copy link
Contributor
si-bors-ng bot commented Aug 28, 2023

try

Build failed:

@zacharyhamm
Copy link
Contributor Author

bors r+

@si-bors-ng
Copy link
Contributor
si-bors-ng bot commented Aug 28, 2023

Build succeeded:

@si-bors-ng si-bors-ng bot merged commit 87da5b2 into main Aug 28, 2023
@si-bors-ng si-bors-ng bot deleted the zack/add-kind-to-pkg-metadata-backwards-compat branch August 28, 2023 19:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant
0