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

feat: allow assignment using = #114

Merged
merged 2 commits into from
Apr 7, 2024
Merged

Conversation

sebffischer
Copy link
Collaborator

Partially solves #101

Copy link
Owner

@dgkf dgkf left a comment

Choose a reason for hiding this comment

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

All looks good! Feel free to add your attribution in the changelog if you'd like. Otherwise, this looks like a sensible addition.

Long-term, I'd like to consolidate the assignment operators, but for the foreseeable I think we can support both. While the project is still just trying to build some recognition, I think it's good to keep it looking R-like with the hope of not turn too many people away from the project for having unfamiliar syntax.

src/CHANGELOG.md Outdated Show resolved Hide resolved
Co-authored-by: Doug Kelkhoff <18220321+dgkf@users.noreply.github.com>
@dgkf
Copy link
Owner

dgkf commented Apr 7, 2024

Thanks again!

@dgkf dgkf merged commit c91e5f1 into dgkf:main Apr 7, 2024
12 checks passed
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.

2 participants