Skip to content

Add CTK validation to PR checks and Go tests #232

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

Open
ricardozanini opened this issue Mar 27, 2025 · 1 comment
Open

Add CTK validation to PR checks and Go tests #232

ricardozanini opened this issue Mar 27, 2025 · 1 comment
Labels
enhancement 🙏 New feature or request spec:1.0.0

Comments

@ricardozanini
Copy link
Member

What would you like to be added:
We have a few feature files defined in the specification:
https://github.com/serverlessworkflow/specification/blob/main/ctk/features/

The reference implementation can now run a few. Currently, we are fetching the spec repository during PR checks to validate the parser. We can do the same for the features to validate the implementation.

Our implementation does not support all the features in the repo so that we might need a filter for the ones supported.

Why is this needed:
To improve the quality of our code base.

@ricardozanini ricardozanini added the enhancement 🙏 New feature or request label Mar 27, 2025
Copy link

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement 🙏 New feature or request spec:1.0.0
Projects
None yet
Development

No branches or pull requests

1 participant