Skip to content

enh: params gen all label force#68

Merged
shinnokdisengir merged 1 commit intoastarte-platform:mainfrom
shinnokdisengir:fix/param-gen-all-explicit-param
Sep 15, 2025
Merged

enh: params gen all label force#68
shinnokdisengir merged 1 commit intoastarte-platform:mainfrom
shinnokdisengir:fix/param-gen-all-explicit-param

Conversation

@shinnokdisengir
Copy link
Copy Markdown
Collaborator

@shinnokdisengir shinnokdisengir commented Sep 4, 2025

enh: params gen all label force

  • params gen all can decostruction destination, seems like gen all (parity feature).
  • added :label in params gen all before decostruction, to hooks the param. It wins over variable name.
  • refactoring.
  • documentation.
  • coverage 100%

@shinnokdisengir shinnokdisengir self-assigned this Sep 4, 2025
@shinnokdisengir shinnokdisengir added the enhancement New feature or request label Sep 4, 2025
@shinnokdisengir shinnokdisengir force-pushed the fix/param-gen-all-explicit-param branch from c43b4d5 to 3758ef5 Compare September 5, 2025 12:42
@shinnokdisengir shinnokdisengir marked this pull request as ready for review September 5, 2025 12:44
@shinnokdisengir shinnokdisengir force-pushed the fix/param-gen-all-explicit-param branch 4 times, most recently from f40f7fa to af86f17 Compare September 8, 2025 11:27
@shinnokdisengir shinnokdisengir force-pushed the fix/param-gen-all-explicit-param branch from af86f17 to 322429a Compare September 9, 2025 09:26
- [x] `params gen all` can decostruction destination, seems like `gen all` but...
- [x] added `:label` in `params gen all` before decostruction (mandatory), to hooks the param. It wins over variable name.
- [x] refactoring.
- [x] documentation.
- [x] coverage 100%

Signed-off-by: Gabriele Ghio <gabriele.ghio@secomind.com>
@shinnokdisengir shinnokdisengir force-pushed the fix/param-gen-all-explicit-param branch from 322429a to eb9fb04 Compare September 9, 2025 09:39
@shinnokdisengir shinnokdisengir merged commit cdcc8f1 into astarte-platform:main Sep 15, 2025
5 checks passed
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