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

Extract ns with render-fns and refer to them by full name #237

Merged
merged 14 commits into from
Oct 21, 2022
Merged

Conversation

mk
Copy link
Member

@mk mk commented Oct 21, 2022

This is a first big step towards making the render-fn part of the viewer api more self-explanatory. We're splitting up the sci-viewer into sci-env and render. Also consistently prefix all render functions and refer to them in viewer using their fully-qualified names so folks can actually jump to definition.

Also start getting rid of most confusing where the exposed var in the sci context would carry a different name than in the defining ns. More cleanup will still follow here.

@mk mk changed the title Move render fns to its own namespace and refer to them by full name Extract ns with :render-fns and refer to them by full name Oct 21, 2022
@mk mk changed the title Extract ns with :render-fns and refer to them by full name Extract ns with render-fns and refer to them by full name Oct 21, 2022
@mk mk merged commit 98059a8 into main Oct 21, 2022
@mk mk deleted the render-reorg branch October 21, 2022 21:51
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.

1 participant