-
Notifications
You must be signed in to change notification settings - Fork 334
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
Support kernels in unsaved files #1540
Conversation
- Clarify multilanguage file kernel logic - Fix flow error - xfail kernel monitor specs, will fix in #1537
lib/store/index.js
Outdated
} | ||
|
||
if (!this.filePath) return null; | ||
// |
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.
Empty comment?
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.
Invisible comment 🧙♂️. We can remove it with magic.
if (!embeddedScope) return grammar; | ||
const scope = embeddedScope.replace(".embedded", ""); | ||
const grammarForScopeName = atom.grammars.grammarForScopeName(scope); | ||
return atom.grammars.grammarForScopeName(scope); |
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 grammarForScopeName
used?
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.
Nope, good catch. We could drop the line above or use the variable 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.
either is fine
@kylebarron you can push those changes if you want and merge. |
…/nteract/hydrogen into support-kernels-in-unsaved-files
Great to have this back, thanks Kyle! |
@kylebarron and I have been collaborating on this.
The branch is ready for a spin if you want to try it. My plan is just to let this bake a few days in case of comments/thoughts/ideas/issues we think of.