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

Configuration overhaul: define default values and remove default values from config file templates #604

Merged
merged 2 commits into from
May 24, 2024
Merged

Configuration overhaul: define default values and remove default values from config file templates #604

merged 2 commits into from
May 24, 2024

Conversation

josecelano
Copy link
Member

@josecelano josecelano commented May 24, 2024 •
edited
Loading

Configuration overhaul: define default values and remove default values from config file templates.

  • Add default values.
  • Remove default values from config file templates.

@josecelano josecelano self-assigned this May 24, 2024
@josecelano josecelano linked an issue May 24, 2024 that may be closed by this pull request
Configuration overhaul: define default values and remove default values from config file templates #603
Closed
@josecelano
refactor: [torrust#603] remove default values from config toml templates
5725d7d 
Not it's possible to specify only values when you wan to overrride
default values.
@josecelano
Copy link
Member Author

ACK 5725d7d

@josecelano josecelano marked this pull request as ready for review May 24, 2024 11:26
@josecelano josecelano requested a review from da2ce7 May 24, 2024 11:26
@josecelano josecelano merged commit 69dd3d3 into torrust:develop May 24, 2024
11 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@da2ce7 da2ce7 Awaiting requested review from da2ce7

Assignees

@josecelano josecelano

Labels
Enhancement / Feature Request Something New
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Configuration overhaul: define default values and remove default values from config file templates
1 participant
@josecelano