Skip to content

Hyphen to snake-case util function #12463

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 1 commit into from
Apr 9, 2025
Merged

Hyphen to snake-case util function #12463

merged 1 commit into from
Apr 9, 2025

Conversation

gentris
Copy link
Contributor

@gentris gentris commented Apr 1, 2025

Motivation

When working with file names, I've encountered scenarios where hyphenated strings (for example: file-name) need to be converted into snake_case (for example: file_name) representations. To standardize processing and improve consistency, I added a simple util function that does that.

Changes

Added hyphen_to_snake_case function

Copy link
Collaborator

@localstack-bot localstack-bot left a comment

Choose a reason for hiding this comment

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

Welcome to LocalStack! Thanks for raising your first Pull Request and landing in your contributions. Our team will reach out with any reviews or feedbacks that we have shortly. We recommend joining our Slack Community and share your PR on the #community channel to share your contributions with us. Please make sure you are following our contributing guidelines and our Code of Conduct.

@localstack-bot
Copy link
Collaborator

localstack-bot commented Apr 1, 2025

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

Copy link

github-actions bot commented Apr 1, 2025

LocalStack Community integration with Pro

    2 files  ±0      2 suites  ±0   1h 53m 43s ⏱️ + 2m 17s
4 318 tests ±0  3 990 ✅ ±0  328 💤 ±0  0 ❌ ±0 
4 320 runs  ±0  3 990 ✅ ±0  330 💤 ±0  0 ❌ ±0 

Results for commit 3921c4a. ± Comparison against base commit ac0ff24.

@gentris gentris added priority: low python Pull requests that update Python code labels Apr 1, 2025
@gentris
Copy link
Contributor Author

gentris commented Apr 3, 2025

I have read the CLA Document and I hereby sign the CLA

localstack-bot added a commit that referenced this pull request Apr 3, 2025
@gentris gentris added semver: patch Non-breaking changes which can be included in patch releases and removed priority: low python Pull requests that update Python code labels Apr 3, 2025
@gentris gentris requested a review from localstack-bot April 3, 2025 00:33
@gentris gentris self-assigned this Apr 3, 2025
@gentris gentris requested review from simonrw and thrau April 7, 2025 12:17
Copy link
Member

@thrau thrau left a comment

Choose a reason for hiding this comment

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

LGTM! thanks for patiently waiting to get it merged ;-)

@thrau thrau merged commit 755eec9 into master Apr 9, 2025
39 of 42 checks passed
@thrau thrau deleted the hyphen-to-snake-case branch April 9, 2025 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver: patch Non-breaking changes which can be included in patch releases
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants