Skip to content

Conversation

@antoniivanov
Copy link
Contributor

@antoniivanov antoniivanov commented Mar 31, 2023

As outlined here -
https://github.com/vmware/versatile-data-kit/blob/main/cicd/build.sh#L40
the top level projects are supposed to have build.sh that fully builds the project or provides intructions

Adding build.sh for frontend

As outlined here -
https://github.com/vmware/versatile-data-kit/blob/main/cicd/build.sh#L40
- the top level projects are supposed to have build.sh that fully builds
the project or provides intructions

Adding build.sh for frontend

Signed-off-by: Antoni Ivanov <aivanov@vmware.com>
@antoniivanov antoniivanov force-pushed the person/aivanov/frontend branch from c8fa70f to 9fd9ade Compare March 31, 2023 07:23
@ivakoleva
Copy link
Contributor

should we test-cover the build.sh requirement and operability

@antoniivanov
Copy link
Contributor Author

should we test-cover the build.sh requirement and operability

We can rely on install_data_pipelines.sh and install_shared.sh are run in CI pipeline. This is primarily useful for someone for a first time building the project locally.

@antoniivanov antoniivanov merged commit 0b652aa into main Mar 31, 2023
@antoniivanov antoniivanov deleted the person/aivanov/frontend branch March 31, 2023 22:11
yonitoo pushed a commit that referenced this pull request Apr 4, 2023
As outlined here -
https://github.com/vmware/versatile-data-kit/blob/main/cicd/build.sh#L40
the top level projects are supposed to have build.sh that fully builds
the project or provides intructions

Adding build.sh for frontend

Signed-off-by: Antoni Ivanov <aivanov@vmware.com>
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.

6 participants