8000 feat: add countBy function and CountBy component by jrmybtlr · Pull Request #129 · jrmybtlr/usemods · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

feat: add countBy function and CountBy component #129

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 2 commits into from
May 2, 2025
Merged

feat: add countBy function and CountBy component #129

merged 2 commits into from
May 2, 2025

Conversation

jrmybtlr
Copy link
Owner
@jrmybtlr jrmybtlr commented Apr 18, 2025

This pull request introduces a new countBy utility function for counting text elements (e.g., characters, words, sentences, paragraphs, or unique strings).

New Feature: countBy Functionality

  • Utility Function Implementation: Added the countBy function in src/goodies.ts to count text elements based on specified criteria (e.g., characters, words, sentences, paragraphs, or unique strings). The function includes support for case-insensitive unique string counting and handles empty inputs gracefully.

Testing Enhancements

  • Unit Tests: Added comprehensive tests for the countBy function in `src/goodies.test.ts

Codebase Maintenance

  • Dead Code Removal: Removed an unused animateGroup function from src/goodies.ts to reduce clutter and improve maintainability.

Miscellaneous

  • Dependency Update: Added a placeholder docs entry in pnpm-lock.yaml for future documentation purposes.

Copy link
Contributor
nuxthub-admin bot commented Apr 18, 2025

❌ Deployment failed usemods

Failed to deploy usemods 61a5131 to preview

📋 View deployment logs

Copy link
cloudflare-workers-and-pages bot commented Apr 18, 2025

Deploying usemods with  Cloudflare Pages  Cloudflare Pages

Latest commit: d73f16e
Status:🚫  Build failed.

View logs

@jrmybtlr jrmybtlr merged commit 8cb9d3b into main May 2, 2025
8 of 9 checks passed
@jrmybtlr jrmybtlr deleted the countby branch May 2, 2025 10:49
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.

1 participant
0