Skip to content

Add STRCOUNT string function #360

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

Merged
merged 13 commits into from
Jun 13, 2025
Merged

Add STRCOUNT string function #360

merged 13 commits into from
Jun 13, 2025

Conversation

john-sanchez31
Copy link
Contributor

Resolves #356

@hadia206
Copy link
Contributor

Make sure to have a commit with [run CI] as part of its message to trigger CI tests on the PR to run.

Copy link
Contributor

@knassre-bodo knassre-bodo left a comment

Choose a reason for hiding this comment

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

Some comments to revise, plus still needs documentation, otherwise LGTM.

Copy link
Contributor

@knassre-bodo knassre-bodo left a comment

Choose a reason for hiding this comment

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

@john-sanchez31 excellent job! Just a few comments to address before merging into main. Also don't forget to click "ready for review"

Copy link
Contributor

@knassre-bodo knassre-bodo left a comment

Choose a reason for hiding this comment

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

LGTM

@knassre-bodo knassre-bodo marked this pull request as ready for review June 13, 2025 16:54
@john-sanchez31 john-sanchez31 merged commit c86be15 into main Jun 13, 2025
5 checks passed
@john-sanchez31 john-sanchez31 deleted the John/str_count branch June 13, 2025 17:33
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.

Add STRCOUNT function to PyDough
3 participants