Replies: 4 comments 1 reply
-
Interesting. I agree that there's no way to do this today, but this seems like something that we should be able to improve. The team is working on a handful of other things right but, but I'll talk to the team about it when they free up. Thanks for the feedback! |
Beta Was this translation helpful? Give feedback.
1 reply
-
Any update on this? |
Beta Was this translation helpful? Give feedback.
0 replies
-
Would be a great feature |
Beta Was this translation helpful? Give feedback.
0 replies
-
Any update? |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
My question is:
When I’m using a reusable workflow, how can I specify using the
needs
parameter in an independent job that I need a specific job of a reusable workflow, not the whole workflow?Let me explain:
I’m using a reusable workflow to store common steps. I have a different job that depends on an inner job of that reusable workflow. I can not specify that I need a specific job of the reusable workflow. All that I can do is refer to the whole reusable workflow.
This is the current image of the workflow:
![image](https://user-images.githubusercontent.com/7026066/152658764-e77af413-204c-4fa1-987d-b98750469389.png)
On the dependent job, I put needs:
pr-test
, to wait forpr-test/lint
because I can not find another way of telling my dependant job to only wait for a specific job of the reusable workflow.Seems natural to me to put
pr-test/lint
, but it doesn’t work. I tried many other configurations without success. It keeps telling me that Job ‘pr-analysis’ depends on unknown job ‘pr-test/lint’.This is what I’m looking for:
![image](https://user-images.githubusercontent.com/7026066/152658771-c6ff206f-af82-441e-8bbe-467f1591d8e1.png)
Here is a try: build: fix sonarqube coverage · bikecoders/ngx-deploy-npm@8402bd9 · GitHub 1
here is the yml simplified:
Note:
What I’m genuinely trying to achieve is to only wait for
pr-test/unit-test(16)
, but the matrix seems to add an extra layer of complexity. That’s why I first tried something more straightforward.Beta Was this translation helpful? Give feedback.
All reactions