Skip to content

Remove parameter_definition_id from NodeParameter input#925

Merged
Taucher2003 merged 1 commit intomainfrom
911-parameterdefinitionid-is-still-required-for-nodeparameter-on-flowinput
Apr 18, 2026
Merged

Remove parameter_definition_id from NodeParameter input#925
Taucher2003 merged 1 commit intomainfrom
911-parameterdefinitionid-is-still-required-for-nodeparameter-on-flowinput

Conversation

@Taucher2003
Copy link
Copy Markdown
Member

@Taucher2003 Taucher2003 commented Apr 16, 2026

Closes #911
Closes #912

@Taucher2003 Taucher2003 self-assigned this Apr 16, 2026
@github-actions
Copy link
Copy Markdown

GitLab Pipeline Action

General information

Link to pipeline: https://gitlab.com/code0-tech/development/sagittarius/-/pipelines/2458794142

Status: Passed
Duration: 6 minutes

Job summaries

rspec: [ee]

Coverage report available at https://code0-tech.gitlab.io/-/development/sagittarius/-/jobs/13960521402/artifacts/tmp/coverage/index.html
Test summary available at https://gitlab.com/code0-tech/development/sagittarius/-/pipelines/2458794142/test_report
Finished in 24.82 seconds (files took 14.78 seconds to load)
1345 examples, 0 failures
Line Coverage: 92.84% (4383 / 4721)
[TEST PROF INFO] Time spent in factories: 00:13.415 (41.12% of total time)

rspec: [ce]

Coverage report available at https://code0-tech.gitlab.io/-/development/sagittarius/-/jobs/13960521401/artifacts/tmp/coverage/index.html
Test summary available at https://gitlab.com/code0-tech/development/sagittarius/-/pipelines/2458794142/test_report
Finished in 25.85 seconds (files took 10.26 seconds to load)
1298 examples, 0 failures
Line Coverage: 92.31% (4215 / 4566)
[TEST PROF INFO] Time spent in factories: 00:13.917 (44.51% of total time)

docs:preview

Documentation preview available at https://code0-tech.gitlab.io/-/development/telescopium/-/jobs/13960558169/artifacts/out/index.html

rubocop

743 files inspected, no offenses detected

@Taucher2003 Taucher2003 merged commit b5d5dce into main Apr 18, 2026
1 check passed
@Taucher2003 Taucher2003 deleted the 911-parameterdefinitionid-is-still-required-for-nodeparameter-on-flowinput branch April 18, 2026 21:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Number value is parsed in as string ParameterDefinitionId is still required for NodeParameter on FlowInput

1 participant