-
Notifications
You must be signed in to change notification settings - Fork 21
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
Gutter items #2980
Gutter items #2980
Conversation
OAGr
commented
Jan 18, 2024
* main: comments fix circular imports by storing tags as Values minor formatting changes split value classes to separate files
🦋 Changeset detectedLatest commit: 76dc633 The changes in this PR will be included in the next version bump. This PR includes changesets to release 6 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Ignored Deployment
|
return uniqBy(arr.reverse(), fn).reverse(); | ||
} | ||
|
||
function visibleAstChildren(node: ASTNode): ASTNode[] { |
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 decided not to merge this into squiggle-lang, because it really is specific to the components
view. It hides values with @hide tags, for example. It's also not too much code.
) | ||
.map((s) => s.location.start.line - 1) | ||
); | ||
return values; |
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.
Looking back, I don't think this refactor to use the AST was worth it yet - mainly because we're not even getting to use the AST pre-render anyway. This code will be a bit more buggy than going straight to values, as it won't always align perfectly.
…ches Gutter items patches