-
Notifications
You must be signed in to change notification settings - Fork 214
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
refactor: use project graph data structure directly #2326
Conversation
☁️ Nx Cloud ReportCI is running/has finished running commands for commit b99dbdb. As they complete they will appear below. Click to see the status, the terminal output, and the build insights. 📂 See all runs for this CI Pipeline Execution ✅ Successfully ran 1 targetSent with 💌 from NxCloud. |
a8577a3
to
d3e5eff
Compare
return nxWorkspace.projectGraph | ||
?.nodes | ||
?.values | ||
?.flatMap { p -> p.data.targets.keys.map { it to p.name } } |
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.
I assume this handles if data and targets is null?
return nxWorkspace.workspace.projects | ||
.filter { it.value.targets.contains(targetsListNode.targetName) } | ||
return nxWorkspace.projectGraph.nodes | ||
.filter { it.value.data.targets.contains(targetsListNode.targetName) } |
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.
Similar question to above
?.entries | ||
?.map { entry -> entry.key to (entry.value.targets.keys) } | ||
?.map { entry -> entry.key to (entry.value.data.targets.keys) } |
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.
Same as above
libs/shared/utils/src/lib/utils.ts
Outdated
w.projects = Object.fromEntries(sortedProjects); | ||
return w; | ||
} | ||
// export function sortWorkspaceProjects( |
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.
Can this commenter section be removed?
d3e5eff
to
66c8285
Compare
80e3bbd
to
b99dbdb
Compare
No description provided.