Skip to content

Conversation

@chenkasirer
Copy link
Member

@Licini how do you feel about adding a pre-build hook task here that will be triggered in https://github.com/compas-dev/compas-actions.build running invoke test and invoke lint.

The reason this would be useful is that now with compas_pb we need to generate the protobuf files before the tests can be run, since these are imported in the library.

so to run the tests, we need:

  1. run the task that compiles the protobuf messages
  2. invoke test

so currently I have to override the compas build action and do it manually.

if we have this "hook", any package that need to do stuff before running the test can implement it..

dunno if there's maybe a native solution in github actions or otherwise.

@Licini
Copy link
Collaborator

Licini commented Oct 29, 2025

@chenkasirer sure we can add a flag to run prebuild option. What would be the command to trigger prebuild? invoke prebuild?

@chenkasirer
Copy link
Member Author

@chenkasirer sure we can add a flag to run prebuild option. What would be the command to trigger prebuild? invoke prebuild?

I think invoke pre-build is cool, no?

@chenkasirer chenkasirer requested a review from Licini October 30, 2025 13:19
@chenkasirer chenkasirer merged commit a9046dc into main Nov 3, 2025
11 checks passed
@chenkasirer chenkasirer deleted the pre-build branch November 3, 2025 16:33
@chenkasirer
Copy link
Member Author

@Licini thanks! will you add it to the build action?

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.

3 participants