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

CI: Add rtdv3 verify callable reusable workflow #32

Closed
wants to merge 1 commit into from

Conversation

jwagantall
Copy link
Contributor

No description provided.

Signed-off-by: Jessica Wagantall <jwagantall@linuxfoundation.org>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

Copy link
Member

@tykeal tykeal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Two things:

  1. Repo linting should be something that is done in repo specific GHA IMO
  2. This job should be done in the project specific .github repo using a workflow_dispatch job handler and not a workflow_call handler (which is used by reusable workflows). You already have an RTD verify reusuable workflow, but you're trying to add it into the testing in ONAP right now.

.github/workflows/composed-rtdv3-verify.yaml Show resolved Hide resolved
@jwagantall
Copy link
Contributor Author

Thanks Andy, yes.. for now we are just adding it to the doc repo in ONAP to confirm it works, but eventually this workflow should live in the .github repo

I wonder if I am making a redundant move by creating this workflow here actually.. but let me move it

@jwagantall jwagantall closed this Aug 15, 2023
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