Skip to content

Add Ripper.tokenize to translation layer#3854

Merged
Earlopain merged 1 commit into
ruby:mainfrom
Earlopain:ripper-tokenize
Jan 16, 2026
Merged

Add Ripper.tokenize to translation layer#3854
Earlopain merged 1 commit into
ruby:mainfrom
Earlopain:ripper-tokenize

Conversation

@Earlopain

@Earlopain Earlopain commented Jan 16, 2026

Copy link
Copy Markdown
Collaborator

It's public API and trivial to implement.

Ref #3838

It's public API and trivial to implement.
@Earlopain

Copy link
Copy Markdown
Collaborator Author

@kddnewton would it be possible to merge my own PRs without review? For some like this where I'm sure it's OK I would like to merge after a day or two if no one approved yet. I guess someone will approve this soon but I'd like to have the option.

@kddnewton

Copy link
Copy Markdown
Collaborator

Yeah feel free. I'm on vacation this week which is why I've been slow to respond. I will review when I get back after they're merged and if there's an issue I can take care of it after they're fact.

@Earlopain

Copy link
Copy Markdown
Collaborator Author

Thing is, I can't. I must pass all the merge requirements and can't self-approve. But you can handle this later after you enjoyed your vacation, I don't want to distract you from your time off.

@kddnewton

Copy link
Copy Markdown
Collaborator

No problem I just made you an admin so you should be able to now.

@Earlopain

Copy link
Copy Markdown
Collaborator Author

Yup, thanks 👍

@Earlopain
Earlopain merged commit 636353b into ruby:main Jan 16, 2026
66 checks passed
@Earlopain
Earlopain deleted the ripper-tokenize branch February 20, 2026 12:42
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