Skip to content

Conversation

@SumoSourabh
Copy link
Contributor

Context:

To reduce the build process, we are adding a build_project.sh files which will help developers to build and test the project via command line options.

Problem:

To build the project, a developer have to go each directory and run two commands to build the project. As we have 5 python packages, so in total commands have to be run 5 times in total to build the full project. Also, to test the project one more command needs to be run on the folder to run all the test cases.

Solution:

Added build_project.sh file that will help developer to build and test the project with simpler commands like sh build_project -bt to build all 5 python modules in one go.

Testing

Tested the script by running various script options for all modules.

Deployment Plan:

No Deployment needed as it is for developers purposes only.

Bonus:

  • Also, updated the pre-commit.yml to only build and test the modules using python 3.8.
  • Also, reformatted the publish-docs.yml.
  • Reformatted how we test the engine api, dvp api version and dvp version using constants. This will help remove merge issues in future.

@SumoSourabh SumoSourabh requested a review from a team as a code owner September 14, 2022 12:22
SumoSourabh and others added 4 commits September 15, 2022 17:48
@SumoSourabh SumoSourabh merged commit 97d163e into delphix:develop Oct 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants