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

Model validation GitHub action will not work with PR changes if PR is from any fork #188

Closed
richaashara opened this issue May 30, 2023 · 3 comments

Comments

@richaashara
Copy link

richaashara commented May 30, 2023

Hello Team,

  1. In Governance.yml file there is a step for model validation action. in its JS file there is a step to validate model file using below command.
    bamm aspect <<model_file_name>> validate
    This can only work if the check out repo contains the model file. But, in-case of PR, repository folder will not have the model file, So, won't it fail in all case?
@richaashara richaashara changed the title Model validation GitHub action will not work with PR changes if PR is from any fork Model validation GitHub action will not work with PR changes if PR is from any fork and also if meta model is samm: <urn:samm:org.eclipse.esmf.samm:meta-model:2.0.0#> May 30, 2023
@richaashara richaashara changed the title Model validation GitHub action will not work with PR changes if PR is from any fork and also if meta model is samm: <urn:samm:org.eclipse.esmf.samm:meta-model:2.0.0#> Model validation GitHub action will not work with PR changes if PR is from any fork May 31, 2023
@bs-jokri
Copy link
Contributor

bs-jokri commented Jul 6, 2023

Hi @richaashara,
thanks for reaching out. As a GitHub action is ran in the context of the branch it is contained in, the validation action should work also for PRs.
you can see this eg in
#204
where the validation is running in a pr.

however, what is still under construction is the move to SAMM. For that we have a PR, which also needs to update the CLI used in order to support also SAMM.

#215

HTH
Johannes

@bs-jokri
Copy link
Contributor

@richaashara does my comment resolve your issue? If so, can we close?

@richaashara
Copy link
Author

yes. @bs-jokri. We can close this issue.

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

No branches or pull requests

2 participants