Skip to content

SONARPY-380 Add parent and children to Python Tree, migrate EmptyNestedBlockCheck, PreIncrementDecrementCheck#275

Merged
andrea-guarino-sonarsource merged 3 commits intomasterfrom
SONARPY-380d
Aug 30, 2019
Merged

SONARPY-380 Add parent and children to Python Tree, migrate EmptyNestedBlockCheck, PreIncrementDecrementCheck#275
andrea-guarino-sonarsource merged 3 commits intomasterfrom
SONARPY-380d

Conversation

@andrea-guarino-sonarsource
Copy link
Copy Markdown
Contributor

No description provided.

@andrea-guarino-sonarsource andrea-guarino-sonarsource changed the title SONARPY-380 Add parent and children to Python Tree SONARPY-380 Add parent and children to Python Tree, migrate EmptyNestedBlockCheck Aug 29, 2019
@andrea-guarino-sonarsource andrea-guarino-sonarsource changed the title SONARPY-380 Add parent and children to Python Tree, migrate EmptyNestedBlockCheck SONARPY-380 Add parent and children to Python Tree, migrate EmptyNestedBlockCheck, PreIncrementDecrementCheck Aug 29, 2019
assertThat(raiseStatement.fromKeyword().getValue()).isEqualTo("from");
assertThat(raiseStatement.fromExpression()).isNotNull();
assertThat(raiseStatement.expressions()).hasSize(1);
assertThat(raiseStatement.children()).hasSize(2);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is that change intended?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I would not add this method to Tree, only to PyStatementListTree, at least for the moment.

@andrea-guarino-sonarsource andrea-guarino-sonarsource merged commit bc5d0a0 into master Aug 30, 2019
@andrea-guarino-sonarsource andrea-guarino-sonarsource deleted the SONARPY-380d branch August 30, 2019 08:05
hashicorp-vault-sonar-prod Bot pushed a commit that referenced this pull request May 26, 2025
Co-authored-by: Thomas Serre <118730793+thomas-serre-sonarsource@users.noreply.github.com>
GitOrigin-RevId: 3f7a97bc58dd2d5d30df1e4bc82e3755b466c477
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.

3 participants