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

Workspaces Part I #1160

Merged
merged 11 commits into from
Jan 16, 2025
Merged

Workspaces Part I #1160

merged 11 commits into from
Jan 16, 2025

Conversation

rydrman
Copy link
Collaborator

@rydrman rydrman commented Dec 17, 2024

This PR is intended to be the first of a few related to #1121

This first PR introduces the spk-workspace crate that defines the structure of a workspace.spk.yaml file and updates the various spk commands to recognize it. By default, a "virtual" workspace is created which collects all *.spk.yaml files in the current directory (this is in-line with previous behavior). The only new feature that becomes unlocked by this is that you can create a workspace.spk.yaml that collects files from additional or different paths.

For example, this repository now has a workspace file that makes spk build spk-convert-pip work from anywhere within it, finding and building from the spec file under the packages folder.

@rydrman rydrman requested a review from jrray December 17, 2024 22:49
@rydrman rydrman self-assigned this Dec 17, 2024
@rydrman rydrman requested a review from dcookspi December 18, 2024 16:54
@rydrman rydrman mentioned this pull request Dec 18, 2024
crates/spk-workspace/src/file.rs Show resolved Hide resolved
crates/spk-workspace/src/file_test.rs Outdated Show resolved Hide resolved
Signed-off-by: Ryan Bottriell <[email protected]>
Although the concept is largely hidden (aside from the new flags) this
adds the ability to define a workspace in the current directory and
modify the default behavior of only loading spk.yaml files from the
current directory.

Signed-off-by: Ryan Bottriell <[email protected]>
Co-authored-by: jrray <[email protected]>
Signed-off-by: Ryan Bottriell <[email protected]>
@rydrman rydrman merged commit 9c856ab into main Jan 16, 2025
5 checks passed
@rydrman rydrman deleted the workspaces branch January 16, 2025 17:55
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