Skip to content

DYN-5128: Fix nodes executing twice after wire reconnect #17095

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
RobertGlobant20 wants to merge 9 commits into
base: master
Choose a base branch
from

Potential fix for pull request finding 'Missed opportunity to use Where'

0b1b423
Select commit
Loading
Failed to load commit list.
Open

DYN-5128: Fix nodes executing twice after wire reconnect #17095

Potential fix for pull request finding 'Missed opportunity to use Where'
0b1b423
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded May 5, 2026 in 1m 8s

Quality Gate passed

Issues
2 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Annotations

Check warning on line 706 in src/Engine/ProtoScript/Runners/LiveRunner.cs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Collection-specific "Contains" method should be used instead of the "Any" extension. 

See more on https://sonarcloud.io/project/issues?id=DynamoDS_Dynamo&issues=AZ36Qja6Xt_5NwazsjEl&open=AZ36Qja6Xt_5NwazsjEl&pullRequest=17095

Check warning on line 704 in src/Engine/ProtoScript/Runners/LiveRunner.cs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Loops should be simplified using the "Where" LINQ method 

See more on https://sonarcloud.io/project/issues?id=DynamoDS_Dynamo&issues=AZ36Qja6Xt_5NwazsjEm&open=AZ36Qja6Xt_5NwazsjEm&pullRequest=17095
View more details on SonarQubeCloud