Skip to content
This repository has been archived by the owner on Jul 17, 2024. It is now read-only.

added empty string matcher #178

Merged
merged 1 commit into from
Dec 5, 2022
Merged

added empty string matcher #178

merged 1 commit into from
Dec 5, 2022

Conversation

includealex
Copy link
Contributor

@includealex
Copy link
Contributor Author

@Graur please take a look

Graur
Graur approved these changes Dec 5, 2022
View reviewed changes
Copy link
Contributor

@Graur Graur left a comment

Choose a reason for hiding this comment

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

@includealex Good one! Thanks!

@Graur
Copy link
Contributor

Graur commented Dec 5, 2022

@rultor merge

@rultor
Copy link
Contributor

rultor commented Dec 5, 2022

@rultor merge

@Graur Unfortunately, either you or me don't have 'write' permissions to the repository, I can't do what you're asking. Try to add me as a collaborator with 'write' access.

@Graur
Copy link
Contributor

Graur commented Dec 5, 2022

@yegor256 please, have a look

@yegor256
Copy link
Member

yegor256 commented Dec 5, 2022

@rultor merge

@rultor
Copy link
Contributor

rultor commented Dec 5, 2022

@rultor merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor rultor merged commit c2dba44 into objectionary:master Dec 5, 2022
@Graur
Copy link
Contributor

Graur commented Dec 5, 2022

@includealex thanks!

@rultor
Copy link
Contributor

rultor commented Dec 5, 2022

@rultor merge

@yegor256 Done! FYI, the full log is here (took me 4min)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@Graur Graur Graur approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@includealex @Graur @rultor @yegor256