Skip to content

Refactor some nodes to remove redundant nodes #1352

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.

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

Merged
merged 14 commits into from
Nov 30, 2023
Merged

Conversation

konradweiss
Copy link
Collaborator

No description provided.

@oxisto oxisto force-pushed the refactor/merge-nodes branch from ac6ba1f to 6a8c626 Compare November 29, 2023 14:51
@oxisto oxisto marked this pull request as ready for review November 29, 2023 19:20
@oxisto oxisto self-requested a review as a code owner November 29, 2023 19:20
@oxisto oxisto enabled auto-merge (squash) November 30, 2023 12:04
konradweiss and others added 14 commits November 30, 2023 13:04

Verified

This commit was signed with the committer’s verified signature. The key has expired.
NeQuissimus Tim Steinbach

Verified

This commit was signed with the committer’s verified signature. The key has expired.
NeQuissimus Tim Steinbach
…his() and super() with construct expressions to the respective records

Verified

This commit was signed with the committer’s verified signature. The key has expired.
NeQuissimus Tim Steinbach
…g unneded code

Verified

This commit was signed with the committer’s verified signature. The key has expired.
NeQuissimus Tim Steinbach
…ble usage of an uneven amount of expressions on lhs and rhs, this PR adapts the find target method assuming that when one side contains more expressions than the other, the first expression on the rhs is a tuple to be unwrapped or the first expression on the lhs is a tuple to be written several values too

Verified

This commit was signed with the committer’s verified signature. The key has expired.
NeQuissimus Tim Steinbach

Verified

This commit was signed with the committer’s verified signature. The key has expired.
NeQuissimus Tim Steinbach

Verified

This commit was signed with the committer’s verified signature. The key has expired.
NeQuissimus Tim Steinbach

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@oxisto oxisto force-pushed the refactor/merge-nodes branch from e9ce128 to d4d1088 Compare November 30, 2023 12:04
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 2 Code Smells

75.2% 75.2% Coverage
0.0% 0.0% Duplication

@oxisto oxisto merged commit 6592ce4 into main Nov 30, 2023
@oxisto oxisto deleted the refactor/merge-nodes branch November 30, 2023 12:23
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.

None yet

2 participants