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 Starlet to the Users section in users.md #277

Merged
merged 2 commits into from
May 7, 2024

Conversation

vt128
Copy link
Contributor

@vt128 vt128 commented May 1, 2024

This PR aims to add Starlet to the list of Starlark users.

Starlet is a Go wrapper for the official Starlark in Go project, designed to enhance the Starlark scripting experience with powerful extensions and enriched wrappers. It offers an intuitive API for executing Starlark scripts, managing global variables, loading modules, controlling the script execution flow, and handling script outputs. Besides, it provides a simplified data exchange mechanism between Go and Starlark types.

Moreover, Starlet includes a set of custom modules and libraries that extend the functionality of the Starlark language, covering a wide range of use cases, such as file manipulation, HTTP client, JSON/CSV handling, and more.

The addition of Starlet to the list of Starlark users would provide a valuable resource for the community, as it illustrates a practical use of Starlark and offers a toolset that can simplify and enhance the Starlark scripting experience for developers.

Please let me know if there are any questions or if further information is needed.

Thanks!

Copy link

google-cla bot commented May 1, 2024

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@vt128
Copy link
Contributor Author

vt128 commented May 5, 2024

@tetromino @stepancheg @laurentlb Could you please review this PR and consider merging it? Thanks!

@laurentlb
Copy link
Contributor

I've found that I no longer have write access here. Sorry :(
#269 has also been pending for a long time.

cc @meisterT

@vt128
Copy link
Contributor Author

vt128 commented May 6, 2024

I've found that I no longer have write access here. Sorry :( #269 has also been pending for a long time.

cc @meisterT

Ah, thanks for the update, @laurentlb! No worries, I appreciate you taking the time to review it. Let's hope the maintainers can address them soon.

@tetromino tetromino self-requested a review May 6, 2024 17:15
Copy link
Collaborator

@tetromino tetromino left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The "users" section is supposed to be for projects that were created to do some intrinsically useful, non-Starlark-related task but happen to use Starlark as a scripting or config language.

Instead, we started putting things like skycfg, starlark-go-nethttp, starlight, and now starlet there; these are not users, these are libraries intended to be used by the users.

I would suggest adding a new "Libraries and extensions" section between tools and IDEs; and moving the skycfg, starlark-go-nethttp, starlight, and starlet items there.

and moving the skycfg, starlark-go-nethttp, starlight, and starlet items there.
@vt128
Copy link
Contributor Author

vt128 commented May 7, 2024

@tetromino Sounds good, I've updated the PR as per your suggestion.

@tetromino
Copy link
Collaborator

Thanks!

@tetromino tetromino merged commit c8d88c3 into bazelbuild:master May 7, 2024
1 of 2 checks passed
@vt128 vt128 deleted the patch-1 branch May 7, 2024 07:59
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

3 participants