Skip to content

Fix #170: Add announcements for component life-cycle + unit tests.#177

Merged
labordep merged 2 commits into
mainfrom
170-Add-announcers-to-know-when-a-Molecule-Component-is-started-and-stopped
Jun 25, 2024
Merged

Fix #170: Add announcements for component life-cycle + unit tests.#177
labordep merged 2 commits into
mainfrom
170-Add-announcers-to-know-when-a-Molecule-Component-is-started-and-stopped

Conversation

@labordep

@labordep labordep commented Apr 7, 2024

Copy link
Copy Markdown
Member

Minor cleanup.

@labordep labordep added the enhancement New feature or request label Apr 7, 2024
@labordep labordep self-assigned this Apr 7, 2024
@labordep labordep requested a review from ELePors April 7, 2024 17:48
@ELePors

ELePors commented Apr 10, 2024

Copy link
Copy Markdown
Member

We should add a cleanup process to reset all announcement registered about components.

@ELePors ELePors left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We should add a cleanup process for announcement

Comment thread src/Molecule-Tests/MolComponentAnnouncementTest.class.st
@labordep

Copy link
Copy Markdown
Member Author

@ELePors Can you send your code into this branch?

@labordep

Copy link
Copy Markdown
Member Author

@ELePors Can you send your code into this branch?

Up :)

@labordep

Copy link
Copy Markdown
Member Author

@ELePors, @Eliott-Guevel is waiting for your commit. Thanks.

@labordep

Copy link
Copy Markdown
Member Author

👍

@labordep labordep assigned ELePors and unassigned labordep Jun 25, 2024

@labordep labordep left a comment

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.

Ok!

@labordep labordep merged commit 96c46d6 into main Jun 25, 2024
@labordep labordep deleted the 170-Add-announcers-to-know-when-a-Molecule-Component-is-started-and-stopped branch June 25, 2024 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants