Skip to content
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

Jump to bottom

Simplify branches with true or false conditions #379

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Simplify branches with true or false conditions #379

wants to merge 8 commits into from

Conversation

github-actions[bot]
Copy link
Contributor

closes #102

@github-actions github-actions bot added the feature-request New feature or request label Jan 17, 2024
@fnhartmann
Copy link
Collaborator

fnhartmann commented Jan 18, 2024 •
edited
Loading

Branches with true or false conditions will be simplified. If this results in an empty condition, this will also be simplified.

For example

if(a == 5){
    if(false){
        c = 5
        return c
    }
}

would be completely removed.

The default setting for this option is False. It will be discussed next week and maybe changed.

@fnhartmann
Copy link
Collaborator

The default setting for this option is False. It will be discussed next week and maybe changed.

As discussed today, we will set this to True.

@fnhartmann fnhartmann marked this pull request as ready for review January 30, 2024 09:44
@steffenenders steffenenders self-requested a review March 7, 2024 09:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@steffenenders steffenenders Awaiting requested review from steffenenders

Assignees

@fnhartmann fnhartmann

Labels
feature-request New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Simplify branches with true or false conditions
1 participant
@fnhartmann