Skip to content

BUG: No merge semantics on den aspects defined multiple times#406

Closed
vic wants to merge 4 commits intomainfrom
push-muntrpktxyqn
Closed

BUG: No merge semantics on den aspects defined multiple times#406
vic wants to merge 4 commits intomainfrom
push-muntrpktxyqn

Conversation

@vic
Copy link
Copy Markdown
Member

@vic vic commented Apr 8, 2026

@vic vic added the allow-ci allow all CI integration tests label Apr 8, 2026
@vic vic force-pushed the push-muntrpktxyqn branch from 5fddc42 to 66d6c6d Compare April 8, 2026 20:39

# do something for testing
den.aspects.tux.user.description = "The Penguin";
imports = [
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Currently den aspects have no merge semantics, we use only one function definition unlike functionTo.

@vic vic changed the title WIP: DO NOT MERGE. BUG: No merge semantics on den aspects defined multiple times Apr 8, 2026
@vic
Copy link
Copy Markdown
Member Author

vic commented Apr 9, 2026

Fixed by #410

@vic vic closed this Apr 9, 2026
@vic vic deleted the push-muntrpktxyqn branch April 13, 2026 00:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

allow-ci allow all CI integration tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant