Skip to content

fix LcFirst doc#1

Merged
cmilesio merged 1 commit into
goforj:mainfrom
sltu:main
Dec 28, 2025
Merged

fix LcFirst doc#1
cmilesio merged 1 commit into
goforj:mainfrom
sltu:main

Conversation

@sltu

@sltu sltu commented Dec 28, 2025

Copy link
Copy Markdown
Contributor

Noticed lack of .LcFirst().String in doc

@cmilesio

Copy link
Copy Markdown
Member

Hey @sltu - thank you for opening the PR.

What you'll want to do is update lcfirst.go example comments, run ./docs/watcher.sh to regenerate documentation and examples.

@cmilesio

Copy link
Copy Markdown
Member

I'm going to merge this as is, but I will follow up with my suggestions and get it mainline

@codecov-commenter

Copy link
Copy Markdown

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

ℹ️ You can also turn on project coverage checks and project coverage reporting on Pull Request comment

Thanks for integrating Codecov - We've got you covered ☂️

@cmilesio cmilesio merged commit 164dfa1 into goforj:main Dec 28, 2025
1 check 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.

3 participants