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.

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

Correct inconsistent feature status in project summary #82

Merged
merged 1 commit into from
Mar 19, 2025

Conversation

marcosbento
Copy link
Collaborator

With these changes, the ON/OFF status of an option is only determined after all necessary evaluations (i.e. conditions, required packages) are finished.

Related to #79.

@marcosbento marcosbento force-pushed the correct_inconsistent_feature_status branch from 323ec04 to d912120 Compare March 18, 2025 13:19
@marcosbento marcosbento requested a review from wdeconinck March 18, 2025 13:59
@marcosbento
Copy link
Collaborator Author

Hi @wdeconinck . If you could have a look at this PR, your review/feedback would be welcome!

Copy link
Member

@wdeconinck wdeconinck left a comment

Choose a reason for hiding this comment

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

Thanks @marcosbento and thanks for the comprehensive tests!

@wdeconinck wdeconinck merged commit e9c6dd2 into develop Mar 19, 2025
284 checks passed
@marcosbento marcosbento deleted the correct_inconsistent_feature_status branch March 19, 2025 08:29
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.

2 participants