Skip to content

simple_event + test#69

Merged
shinnokdisengir merged 12 commits intoastarte-platform:mainfrom
shinnokdisengir:feature/triggers/simple-event
Oct 21, 2025
Merged

simple_event + test#69
shinnokdisengir merged 12 commits intoastarte-platform:mainfrom
shinnokdisengir:feature/triggers/simple-event

Conversation

@shinnokdisengir
Copy link
Copy Markdown
Collaborator

complete triggers events + simple_event wrapper

@shinnokdisengir shinnokdisengir self-assigned this Sep 18, 2025
@shinnokdisengir shinnokdisengir force-pushed the feature/triggers/simple-event branch 6 times, most recently from 1493078 to e4e8532 Compare September 22, 2025 10:00
@shinnokdisengir shinnokdisengir marked this pull request as ready for review September 23, 2025 10:07
@shinnokdisengir shinnokdisengir added the You shall not pass! Until the issue described is resolved, it should not be merged. label Sep 26, 2025
@shinnokdisengir shinnokdisengir marked this pull request as draft September 26, 2025 10:35
@shinnokdisengir shinnokdisengir force-pushed the feature/triggers/simple-event branch 2 times, most recently from 04bb080 to b10ecba Compare September 29, 2025 09:18
@shinnokdisengir shinnokdisengir force-pushed the feature/triggers/simple-event branch from b10ecba to 5157cc8 Compare October 8, 2025 13:04
@shinnokdisengir shinnokdisengir removed the You shall not pass! Until the issue described is resolved, it should not be merged. label Oct 9, 2025
@shinnokdisengir shinnokdisengir force-pushed the feature/triggers/simple-event branch 4 times, most recently from 1d70da3 to 9499f5d Compare October 13, 2025 15:41
@shinnokdisengir shinnokdisengir marked this pull request as ready for review October 13, 2025 15:47
@shinnokdisengir shinnokdisengir force-pushed the feature/triggers/simple-event branch 2 times, most recently from e970c1c to 4c295f5 Compare October 15, 2025 18:35
@shinnokdisengir shinnokdisengir force-pushed the feature/triggers/simple-event branch 4 times, most recently from 1ad0965 to c235c64 Compare October 20, 2025 07:43
Copy link
Copy Markdown
Collaborator

@noaccOS noaccOS left a comment

Choose a reason for hiding this comment

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

second review ronud
some comments are relative to multiple files (eg all events relative to a generated value) but I only left one for each type as a general comment

@shinnokdisengir shinnokdisengir force-pushed the feature/triggers/simple-event branch from c235c64 to 095d865 Compare October 20, 2025 10:15
- value generator feature and fixes + test
- interface generator endpoint_by_aggregation + test
- interface generator minor_version become public

Signed-off-by: Gabriele Ghio <gabriele.ghio@secomind.com>
Signed-off-by: Gabriele Ghio <gabriele.ghio@secomind.com>
Signed-off-by: Gabriele Ghio <gabriele.ghio@secomind.com>
Signed-off-by: Gabriele Ghio <gabriele.ghio@secomind.com>
Signed-off-by: Gabriele Ghio <gabriele.ghio@secomind.com>
Signed-off-by: Gabriele Ghio <gabriele.ghio@secomind.com>
Signed-off-by: Gabriele Ghio <gabriele.ghio@secomind.com>
Signed-off-by: Gabriele Ghio <gabriele.ghio@secomind.com>
Signed-off-by: Gabriele Ghio <gabriele.ghio@secomind.com>
@shinnokdisengir shinnokdisengir force-pushed the feature/triggers/simple-event branch from 095d865 to 4957ee5 Compare October 20, 2025 11:01
Signed-off-by: Gabriele Ghio <gabriele.ghio@secomind.com>
Signed-off-by: Gabriele Ghio <gabriele.ghio@secomind.com>
Signed-off-by: Gabriele Ghio <gabriele.ghio@secomind.com>
@shinnokdisengir shinnokdisengir force-pushed the feature/triggers/simple-event branch from 4957ee5 to 753c502 Compare October 21, 2025 10:22
Copy link
Copy Markdown
Collaborator

@noaccOS noaccOS left a comment

Choose a reason for hiding this comment

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

🚀

@shinnokdisengir shinnokdisengir merged commit d0bd9d9 into astarte-platform:main Oct 21, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants