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

Make every configuration field optional and use default if not passed #48

Merged
merged 3 commits into from
Oct 19, 2023
Merged

Make every configuration field optional and use default if not passed #48

merged 3 commits into from
Oct 19, 2023

Conversation

alt-art
Copy link
Owner

@alt-art alt-art commented Oct 19, 2023

You dont have to pass every field on the configurations file anymore. Now commit as a default for every field and will use them instead of throwing a error about a required field and that's why the init command will be discontinued.

@alt-art
fix(test): Use serial_test to run synchronous tests
86ed8bc 
Now file tests will be done synchronously so as not to mess up with the file system
@alt-art
feat(config): Make every field the default
fcf3291 
You dont have to pass every field on the configurations file anymore. Now commit as a default for every field and will use them instead of throwing a error about a required field and that's why the init command will be discontinued.
@alt-art alt-art added the enhancement New feature or request label Oct 19, 2023
@alt-art alt-art self-assigned this Oct 19, 2023
@alt-art alt-art merged commit 0e4353e into main Oct 19, 2023
7 checks passed
@alt-art alt-art deleted the easy-config branch October 19, 2023 02:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@alt-art alt-art

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

Successfully merging this pull request may close these issues.
1 participant
@alt-art