Skip to content

Add solution for Challenge 6 by kudesn1k1#1464

Merged
RezaSi merged 2 commits intoRezaSi:mainfrom
kudesn1k1:challenge-6-kudesn1k1
Mar 16, 2026
Merged

Add solution for Challenge 6 by kudesn1k1#1464
RezaSi merged 2 commits intoRezaSi:mainfrom
kudesn1k1:challenge-6-kudesn1k1

Conversation

@kudesn1k1
Copy link
Contributor

Challenge 6 Solution

Submitted by: @kudesn1k1
Challenge: Challenge 6

Description

This PR contains my solution for Challenge 6.

Changes

  • Added solution file to challenge-6/submissions/kudesn1k1/solution-template.go

Testing

  • Solution passes all test cases
  • Code follows Go best practices

Thank you for reviewing my submission! 🚀

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 28, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 857c6c1 and ac7fb38.

📒 Files selected for processing (1)
  • challenge-6/submissions/kudesn1k1/solution-template.go

Walkthrough

Adds an exported function CountWordFrequency(text string) map[string]int to package challenge6 that lowercases input, treats apostrophes as ignored, uses non-alphanumeric characters as word boundaries, tokenizes alphanumeric sequences, and returns a map of word counts.

Changes

Cohort / File(s) Summary
Word Frequency Counter
challenge-6/submissions/kudesn1k1/solution-template.go
Adds CountWordFrequency(text string) map[string]int which lowercases input, accumulates alphanumeric rune sequences as words (ignoring apostrophes), treats other non-alphanumerics as separators, and returns frequency counts in map[string]int.

Estimated Code Review Effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly Related PRs

  • #757: Adds the same CountWordFrequency(text string) map[string]int function to package challenge6 with identical signature and purpose.
  • #653: Implements CountWordFrequency in the challenge6 package with similar tokenization and counting logic.
  • #664: Adds CountWordFrequency for word-frequency counting in challenge6 with comparable normalization and parsing.

Poem

🐰 I hopped through text both near and far,
Lowercased letters, skipped the apostrophe star,
Boundaries found where symbols play,
Counting words in a joyful way,
Hooray — the map blooms word-by-word today!

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: adding a solution for Challenge 6 by the specified user.
Description check ✅ Passed The description is directly related to the changeset, providing context about the Challenge 6 submission and referencing the exact file path being added.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🧹 Nitpick comments (1)
challenge-6/submissions/kudesn1k1/solution-template.go (1)

21-31: Inconsistent indentation (mixed tabs and spaces).

Line 21 uses spaces while lines 23-31 use tabs. Running gofmt will normalize the indentation to tabs.


ℹ️ Review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between a7ac764 and 857c6c1.

📒 Files selected for processing (1)
  • challenge-6/submissions/kudesn1k1/solution-template.go

@RezaSi RezaSi merged commit 5e905a0 into RezaSi:main Mar 16, 2026
5 checks passed
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