Skip to content

Revisit supporting namespaces#649

Merged
jasmith-hs merged 9 commits into
masterfrom
namespaces
Apr 30, 2021
Merged

Revisit supporting namespaces#649
jasmith-hs merged 9 commits into
masterfrom
namespaces

Conversation

@jasmith-hs

Copy link
Copy Markdown
Contributor

This PR builds off of @pastorcmentarny's PR #306 with a few modifications:

  • Allows multiple namespaces in any context
  • Allows namespaces to be constructed like in Jinja with an optional dictionary to initialize with and optional keyword arguments
  • Makes Namespace an extension of HashMap<String, Object>
  • Also adds necessary logic for namespaces to work with eager execution.

This PR will close out #274

@jasmith-hs jasmith-hs requested a review from mattcoley April 20, 2021 21:14
@jasmith-hs jasmith-hs requested a review from boulter April 30, 2021 17:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants