Skip to content

Comments

Use Process.label() instead of Process.register(). #1071

Merged
varsill merged 21 commits intomasterfrom
varsill/set_label_for_processes
Feb 12, 2026
Merged

Use Process.label() instead of Process.register(). #1071
varsill merged 21 commits intomasterfrom
varsill/set_label_for_processes

Conversation

@varsill
Copy link
Contributor

@varsill varsill commented Feb 10, 2026

This PR:

  • Uses Process.label() instead of Process.register() inside the Stalker
  • Deprecates :components option for :unsafely_name_processes_for_observer
  • Hard deprecated :links option for :unsafely_name_processes_for_observer in favour of :report_links_to_observer config

closes #1068

@varsill varsill marked this pull request as ready for review February 11, 2026 15:22
@varsill varsill requested a review from mat-hek as a code owner February 11, 2026 15:22
@varsill varsill marked this pull request as draft February 11, 2026 15:39
@varsill varsill self-assigned this Feb 11, 2026
@varsill varsill added this to Smackore Feb 11, 2026
@varsill varsill moved this to In Progress in Smackore Feb 11, 2026
@varsill varsill marked this pull request as ready for review February 12, 2026 08:56
@varsill varsill moved this from In Progress to In Review in Smackore Feb 12, 2026
CHANGELOG.md Outdated
Comment on lines 5 to 6
* Deprecate `:components` option for `:unsafely_name_processes_for_observer`
* Deprecate `:links` option for `:unsafely_name_processes_for_observer` in favour of `:report_links_to_observer` configuration entry
Copy link
Member

Choose a reason for hiding this comment

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

Mention you added the labels ;)

varsill and others added 5 commits February 12, 2026 12:36
Co-authored-by: Mateusz Front <mateusz.front@swmansion.com>
Co-authored-by: Mateusz Front <mateusz.front@swmansion.com>
@varsill varsill merged commit 23d947b into master Feb 12, 2026
5 of 6 checks passed
@varsill varsill deleted the varsill/set_label_for_processes branch February 12, 2026 13:00
@github-project-automation github-project-automation bot moved this from In Review to Done in Smackore Feb 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Set label for Membrane processes

2 participants