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.

Sign up for GitHub

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

Jump to bottom

Cyclic dependency: vs/editor/common/viewLayout/lineDecorations and vs/editor/common/viewLayout/viewLineRenderer #150177

Closed
Tracked by #150025
jrieken opened this issue May 23, 2022 · 0 comments · Fixed by #150259
Closed
Tracked by #150025

Cyclic dependency: vs/editor/common/viewLayout/lineDecorations and vs/editor/common/viewLayout/viewLineRenderer #150177

jrieken opened this issue May 23, 2022 · 0 comments · Fixed by #150259
Assignees
@alexdima
Labels
insiders-released Patch has been released in VS Code Insiders
Milestone
May 2022

Comments

@jrieken
Copy link
Member

jrieken commented May 23, 2022

related #150025

There is a cycling (runtime) dependency when not inlining/hoisting const enums between the mentioned files. The enum in question is: LinePartMetadata
@jrieken jrieken mentioned this issue May 23, 2022
Closed
15 tasks
@jrieken jrieken changed the title Cycling dependency: vs/editor/common/viewLayout/lineDecorations and vs/editor/common/viewLayout/viewLineRenderer Cyclic dependency: vs/editor/common/viewLayout/lineDecorations and vs/editor/common/viewLayout/viewLineRenderer May 23, 2022
@alexdima alexdima mentioned this issue May 24, 2022
Merged
alexdima added a commit that referenced this issue May 24, 2022
@alexdima
Fixes #150177: Extract LinePartMetadata to a separate file
77578ad
@alexdima alexdima added this to the May 2022 milestone May 24, 2022
@VSCodeTriageBot VSCodeTriageBot added the unreleased Patch has not yet been released in VS Code Insiders label May 24, 2022
alexdima added a commit that referenced this issue May 24, 2022
@alexdima
Fixes #150177: Extract LinePartMetadata to a separate file (#150259)
faa736e
@VSCodeTriageBot VSCodeTriageBot added insiders-released Patch has been released in VS Code Insiders and removed unreleased Patch has not yet been released in VS Code Insiders labels May 25, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Jul 8, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@alexdima alexdima

Labels
insiders-released Patch has been released in VS Code Insiders
Projects
None yet
Milestone
May 2022
Development

Successfully merging a pull request may close this issue.

Extract LinePartMetadata to a separate file microsoft/vscode
4 participants
@jrieken @hediet @alexdima @VSCodeTriageBot