conf: Make conf validation recognize AccountSettings extra keys #351
No reviewers
Labels
No Label
IMAP
JMAP
Maildir
Retired
User Experience
User Interface
bsd
bug
contacts
currently worked on
documentation
duplicate
easy
enhancement
help wanted
invalid
linux-gnu
macos
mbox
notmuch
question
security
wishlist
wontfix
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: meli/meli#351
Loading…
Reference in New Issue
There is no content yet.
Delete Branch "fix-349-vcard_folder-bug"
Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?
Summary of the PR
AccountSettings extra keys like
vcard_folder
were not taken intoaccount when validating a config.
This commit introduces an AccountSettings::validate_config() method that
checks for the presence and validity of this key value.
Fixes #349
Requirements
Before submitting your PR, please make sure you have addressed the following requirements:
git commit -s
), and the commit has a message describing the motivation behind the change, if appropriate.unsafe
code is properly documented.rustfmt
. Runmake fmt
in the project root.clippy
. Runmake lint
in the project root.make test
in the project root. If you havecargo-nextest
installed, you can runcargo nextest run --all --no-fail-fast --all-features --future-incompat-report
instead.