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

Restructure valor workspace #52

Closed
wants to merge 15 commits into from
Closed

Restructure valor workspace #52

wants to merge 15 commits into from

Conversation

pandres95
Copy link
Member

@pandres95 pandres95 commented Jun 16, 2023

This PR aims to better separate the logic folder structure of the repository between multiple crate libraries (as it'll be the case in the future), while separating them to the final CLIs/executables, allowing working in a more seamless, collaborative, non-blocking way.

Here we introduce the following folder structure:

- bin/
 |- valor-run
 |- valor-http
 |- ... 
- lib/
 |- core/
 |- runtime/
 |- ...
- Cargo.toml

@pandres95 pandres95 requested review from olanod and johandroid June 16, 2023 08:51
@pandres95 pandres95 self-assigned this Jun 16, 2023
@pandres95 pandres95 added the enhancement New feature or request label Jun 16, 2023
@pandres95 pandres95 added this to the v1.0 milestone Jun 16, 2023
@pandres95 pandres95 linked an issue Jun 16, 2023 that may be closed by this pull request
@pandres95 pandres95 changed the title Redefining Valor Interfaces Restructure valor workspace Jun 20, 2023
@pandres95 pandres95 closed this Jun 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Common interfacing for Valor modules
1 participant