-
Notifications
You must be signed in to change notification settings - Fork 4k
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
chore(pipelines): write a GraphViz file with the pipeline structure #23908
Conversation
Add a `pipeline.dot` file to the cloud assembly containing the graph structure of the pipeline. This change is a `chore`, not a `feat`, as I want this to be a debugging aid, but I don't want to service feature requests on it.
let i = 0; | ||
const sortedNodes = Array.prototype.concat.call([], ...x.sortedChildren()); | ||
for (const child of sortedNodes) { | ||
recurse(child, `${indent} ${follow} `, i++ == x.nodes.size - 1); |
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.
nice one 🙂
Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork). |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork). |
…23908) Add a `pipeline.dot` file to the cloud assembly containing the graph structure of the pipeline. This change is a `chore`, not a `feat`, as I want this to be a debugging aid, but I don't want to service feature requests on it. ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
…23908) Add a `pipeline.dot` file to the cloud assembly containing the graph structure of the pipeline. This change is a `chore`, not a `feat`, as I want this to be a debugging aid, but I don't want to service feature requests on it. ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
…24030) This is a re-roll of #23908 which had to be reverted in #24006 because there are some cases where change set approval steps that are shared between two stacks would cause cyclic dependencies between those stacks that caused them to be mutually unsortable. Solve this by adding a mode to the toposort routine that will proceed even if there are cyclic dependencies (which is used purely for rendering). ---- Add a `pipeline.dot` file to the cloud assembly containing the graph structure of the pipeline. This change is a `chore`, not a `feat`, as I want this to be a debugging aid, but I don't want to service feature requests on it. ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
Add a
pipeline.dot
file to the cloud assembly containing the graph structure of the pipeline.This change is a
chore
, not afeat
, as I want this to be a debugging aid, but I don't want to service feature requests on it.By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license