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.

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

Add syntax highlighting to singular kernel #7

Merged
merged 1 commit into from
Dec 11, 2017

Conversation

vinklein
Copy link
Contributor

@vinklein vinklein commented Dec 1, 2017

Add a codemirror "SimpleMode" to singular kernel
Modify setup.py (previous version doesn't remove installed files under
local/share/jupyter/kernels)

Linked to OpenDreamkit issue #175

@sebasguts
I used mainly this resource https://www.singular.uni-kl.de/Manual/latest/index.htm as base to do this syntax highlight.
Please tell me if it looks up to date for you and if i am missing something important with the
singular syntax.
About type tokens i set them to the "qualifier" kind of token, because "type" kind of token doesn't have a
css style applied with jupyter.

Add a codemirror "SimpleMode" to singular kernel
Modify setup.py (previous version doesn't remove installed files under
local/share/jupyter/kernels)
@sebasguts
Copy link
Owner

Thanks a lot :). I will review the PR this weekend and merge it asap :).

@sebasguts sebasguts merged commit 153193a into sebasguts:master Dec 11, 2017
@sebasguts
Copy link
Owner

I merged your changes and acknowledged them in the README.md. Since the constribution is very substantial, I can also make you an author, if you prefer. Let me know what you would like.

@vinklein
Copy link
Contributor Author

Thanks. I am fine with the acknowledgment.

@sebasguts
Copy link
Owner

All right.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants