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 configure_starlark #34

Merged
merged 1 commit into from
Apr 20, 2022
Merged

Add configure_starlark #34

merged 1 commit into from
Apr 20, 2022

Conversation

jordemort
Copy link
Contributor

starlark.Resolve contains some global boolean values that control how
Starlark behaves; the most interesting of these is AllowSet, which
allows the set data type.

This adds a Python function to allow tweaking these settings. We only
expose the ones that aren't deprecated.

Also:

  • Tweaked the .pyi to properly specify keyword-only arguments
  • Ditched the old statically declared version in init

Ref: https://pkg.go.dev/go.starlark.net/resolve#pkg-variables

starlark.Resolve contains some global boolean values that control how
Starlark behaves; the most interesting of these is AllowSet, which
allows the `set` data type.

This adds a Python function to allow tweaking these settings. We only
expose the ones that aren't deprecated.

Also:
  - Tweaked the .pyi to properly specify keyword-only arguments
  - Ditched the old statically declared version in __init__

Ref: https://pkg.go.dev/go.starlark.net/resolve#pkg-variables
@jordemort jordemort merged commit d19c646 into main Apr 20, 2022
@jordemort jordemort deleted the configure branch April 20, 2022 17:32
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