-
Notifications
You must be signed in to change notification settings - Fork 9
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
feat(1415): Add subscribe field in parsed config #109
feat(1415): Add subscribe field in parsed config #109
Conversation
lib/phase/functional.js
Outdated
@@ -223,6 +223,17 @@ module.exports = async (flattenedDoc, buildClusterFactory, triggerFactory, pipel | |||
|
|||
// Workflow graph | |||
doc.workflowGraph = await generateWorkflowGraph(doc, triggerFactory, pipelineId); | |||
|
|||
doc.workflowGraph.nodes.forEach(node => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this needed because it get's saved with without ~
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah this being getting saved without ~
was causing some problems. I needed this trigger to be the same as ~pr
and ~commit
.
Context
Fixes screwdriver-cd/screwdriver#1415
Objective
This PR adds the
subscribe
field to the parsed config.References
screwdriver-cd/screwdriver#1415
License
I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.