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(utils): add an utility function to find string prefix by separator #1338

Merged
merged 2 commits into from
Feb 3, 2023

Conversation

WHBANG
Copy link
Contributor

@WHBANG WHBANG commented Feb 2, 2023

What problem does this PR solve?

#1337

  • Unit test

@github-actions github-actions bot added the cpp label Feb 2, 2023
@WHBANG WHBANG force-pushed the dev/find_string_prefix_tool branch from fa5c571 to f440ee6 Compare February 2, 2023 10:23
acelyc111
acelyc111 previously approved these changes Feb 2, 2023
@acelyc111 acelyc111 changed the title feat(utils): add a tool method find_string_ prefix in utils feat(utils): add a utility function to find string prefix by separator Feb 2, 2023
@acelyc111 acelyc111 changed the title feat(utils): add a utility function to find string prefix by separator feat(utils): add an utility function to find string prefix by separator Feb 2, 2023
src/utils/strings.cpp Outdated Show resolved Hide resolved
src/utils/strings.cpp Outdated Show resolved Hide resolved
src/utils/test/utils.cpp Show resolved Hide resolved
src/utils/test/utils.cpp Outdated Show resolved Hide resolved
@acelyc111 acelyc111 merged commit 390ccf8 into apache:master Feb 3, 2023
@empiredan empiredan mentioned this pull request Aug 17, 2023
GehaFearless pushed a commit to GehaFearless/incubator-pegasus that referenced this pull request Feb 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants