Skip to content

Hotfix: Fix incorrect coalescing order for normalize_edges#1663

Merged
piotrkan merged 3 commits intomainfrom
hotfix/normalize-coalescion
Jul 8, 2025
Merged

Hotfix: Fix incorrect coalescing order for normalize_edges#1663
piotrkan merged 3 commits intomainfrom
hotfix/normalize-coalescion

Conversation

@piotrkan
Copy link
Copy Markdown
Member

@piotrkan piotrkan commented Jul 8, 2025

Description of the changes

Fixes / Resolves the following issues:

Checklist:

  • Added label to PR (e.g. enhancement or bug)
  • Ensured the PR is named descriptively. FYI: This name is used as part of our changelog & release notes.
  • Looked at the diff on github to make sure no unwanted files have been committed.
  • Made corresponding changes to the documentation
  • Added tests that prove my fix is effective or that my feature works
  • Any dependent changes have been merged and published in downstream modules
  • If breaking changes occur or you need everyone to run a command locally after
    pulling in latest main, uncomment the below "Merge Notification" section and
    describe steps necessary for people
  • Ran on sample data using kedro run -e sample -p test_sample (see sample environment guide)

@piotrkan piotrkan requested a review from a team as a code owner July 8, 2025 09:41
@piotrkan piotrkan requested a review from JacquesVergine July 8, 2025 09:41
@piotrkan piotrkan added the bug Something isn't working / was fixed label Jul 8, 2025
@piotrkan
Copy link
Copy Markdown
Member Author

piotrkan commented Jul 8, 2025

Can confirm it worked witth v0.8.2-prod release

@piotrkan piotrkan merged commit 615fa04 into main Jul 8, 2025
7 checks passed
amyford pushed a commit that referenced this pull request Aug 14, 2025
Co-authored-by: Jacques Vergine <jacques.vergine35@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working / was fixed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants