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

Refactor to make it work under RakuAST #1

Merged
merged 1 commit into from
Sep 26, 2023

Conversation

lizmat
Copy link
Contributor

@lizmat lizmat commented Sep 24, 2023

  • Add dependency to Slangify to abstract the slangification in a central place, to allow for possible future changes behind the scenes.
  • Add preliminary support to allow running under RakuAST. This requires a post 2023.09 version of Rakudo, specifically 9b29fdcab61
  • Add some basic tests, also in "do" context
  • Add .gitignore for convenience
  • Does NOT bump version yet

For the record: thank you for creating this slang after the discussions after Damian's blog post, now 4 years ago. If you think it would be better if this were a Raku community module, transfer the repo to raku-community-modules, and I will take care of it there. Of course, if you would like to keep taking care of this module, that's also ok!

- Add dependency to Slangify to abstract the slangification in a
  central place, to allow for possible future changes behind the
  scenes.
- Add preliminary support to allow running under RakuAST.  This
  requires a post 2023.09 version of Rakudo, specifically 9b29fdcab61
- Add some basic tests, also in "do" context
- Add .gitignore for convenience
- Does NOT bump version yet

For the record: thank you for creating this slang after the discussions
after Damian's blog post, now 4 years ago.  If you think it would be
better if this were a Raku community module, transfer the repo to
raku-community-modules, and I will take care of it there.  Of course,
if you would like to keep taking care of this module, that's also ok!
@0racle 0racle merged commit 8ef73d0 into 0racle:master Sep 26, 2023
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

2 participants