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

Make sets complement each other #19

Merged
merged 2 commits into from
Sep 19, 2024
Merged

Conversation

come-nc
Copy link
Collaborator

@come-nc come-nc commented Sep 19, 2024

Each set should build upon the previous one so that to be compatible with 27 only 27 set is needed.
For PHP they use 'UP_TO_' prefixed sets for that but I think we can simply do that directly in default sets.

I removed the ALL set as I do not see the point. We could have a LATEST but I find that dangerous.

I’m starting to think we’re gonna need to start lower than 25 though, to avoid having an unreadable huge set.

@come-nc come-nc self-assigned this Sep 19, 2024
Copy link
Collaborator

@ChristophWurst ChristophWurst left a comment

Choose a reason for hiding this comment

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

👍

@come-nc come-nc force-pushed the feat/make-sets-complement-each-other branch from 2607c8c to 0cde8b6 Compare September 19, 2024 08:50
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
…n they support

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
@come-nc come-nc force-pushed the feat/make-sets-complement-each-other branch from 0cde8b6 to 6725df4 Compare September 19, 2024 09:43
@come-nc come-nc merged commit c5cceb7 into main Sep 19, 2024
21 checks passed
@come-nc come-nc deleted the feat/make-sets-complement-each-other branch September 19, 2024 09:54
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