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

feat(config): introduce configuration builder #1053

Merged
merged 2 commits into from
May 27, 2024
Merged

Conversation

Altahrim
Copy link
Contributor

@Altahrim Altahrim commented May 16, 2024

Content

Introduce a configuration builder based on JSON files:

  • major_versions.json declare all Nextcloud major versions with end of life and minimal supported PHP version
  • releases.json list all releases with their attributes (signature, internal version number and deploy percentage)

The builder should be able to generate the config.php we are used to.

TODO

  • Add README.md
  • Add check to verify config.php is up-to-date with the JSON files
  • Check generated config.php file (pass tests)
  • Rebase before merge to relaunch tests with latest versions ⚠️

@Altahrim Altahrim self-assigned this May 16, 2024
@Altahrim Altahrim force-pushed the feat/config-builder branch 5 times, most recently from 68d8594 to 6d77807 Compare May 16, 2024 13:27
@Altahrim Altahrim force-pushed the feat/config-builder branch 2 times, most recently from d91c525 to 9f85c31 Compare May 16, 2024 13:43
@Altahrim Altahrim marked this pull request as ready for review May 16, 2024 13:46
@Altahrim Altahrim requested a review from a team as a code owner May 16, 2024 13:46
@joshtrichards
Copy link

Nice!

Since you're already stuffing the branch eol date into the json, this would be an ideal opportunity for #1050.

This would let me do this: nextcloud/server#45402 (comment)

Copy link
Contributor

@kesselb kesselb left a comment

Choose a reason for hiding this comment

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

Cool 👍

Makefile Outdated Show resolved Hide resolved
@Altahrim Altahrim force-pushed the feat/config-builder branch 2 times, most recently from c3b6815 to c32aa58 Compare May 22, 2024 06:42
@Altahrim Altahrim force-pushed the feat/config-builder branch 2 times, most recently from 8ea3d28 to 280a290 Compare May 22, 2024 13:05
@Altahrim
Copy link
Contributor Author

Rebased for latest config.php and fixed code/tests

@Altahrim Altahrim requested a review from blizzz May 22, 2024 13:46
@Altahrim Altahrim force-pushed the feat/config-builder branch 2 times, most recently from db138d4 to 7d8f66f Compare May 27, 2024 14:03
Signed-off-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.com>
Signed-off-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.com>
@Altahrim Altahrim merged commit 30963a5 into master May 27, 2024
23 checks passed
@Altahrim Altahrim deleted the feat/config-builder branch May 27, 2024 14:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

5 participants