kedro icon indicating copy to clipboard operation
kedro copied to clipboard

Feature/normalize data

Open suryasreddy opened this issue 5 months ago • 3 comments

Description

Add a small naming utilities module to core Kedro.

Development notes

  • Pure Python utility with no new dependencies.
  • Backward compatible. Adds a new module only.
  • Unit tests in tests/test_naming.py cover:
    • basic slugging, collapse of underscores, leading digit guard, empty/invalid input
    • uniqueness logic across consecutive collisions
    • type validation
  • Ran tests locally with pytest -q tests/test_naming.py.

Developer Certificate of Origin

We need all contributions to comply with the Developer Certificate of Origin (DCO). All commits must be signed off by including a Signed-off-by line in the commit message. See our wiki for guidance.

If your PR is blocked due to unsigned commits, then you must follow the instructions under "Rebase the branch" on the GitHub Checks page for your PR. This will retroactively add the sign-off to all unsigned commits and allow the DCO check to pass.

Checklist

  • [x] Read the contributing guidelines
  • [x ] Signed off each commit with a Developer Certificate of Origin (DCO)
  • [x ] Opened this PR as a 'Draft Pull Request' if it is work-in-progress
  • [x ] Updated the documentation to reflect the code changes
  • [ ] Added a description of this change in the RELEASE.md file
  • [x ] Added tests to cover my changes
  • [ ] Checked if this change will affect Kedro-Viz, and if so, communicated that with the Viz team

suryasreddy avatar Nov 06 '25 01:11 suryasreddy

@suryasreddy Thanks for your interest in contributing. Why do you think this should be a part of Kedro?

deepyaman avatar Nov 06 '25 02:11 deepyaman

Same question as @deepyaman , also please make sure your venv is gitignored

datajoely avatar Nov 06 '25 11:11 datajoely

Hi @suryasreddy, thanks for your contribution. It's been a while since you opened it. Are you still interested in finishing it? I have the same question as @deepyaman and @datajoely , so would be great to hear your thoughts 🙂

merelcht avatar Dec 02 '25 14:12 merelcht

I'm closing this PR due to inactivity. Feel free to re-open if you want to continue working on this @suryasreddy and let us know if you need any help.

merelcht avatar Dec 16 '25 11:12 merelcht