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

Document commands.conf #64

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Document commands.conf #64

wants to merge 1 commit into from

Conversation

Beherith
Copy link
Contributor

Please check if these are actually correct :)

Please check if these are actually correct :)
@Yaribz
Copy link
Owner

Yaribz commented Oct 29, 2023

Thank you for your pull request !

Actually several SPADS config files share the same syntax, based on a list of usable input fields (filters), dynamic customizable templates (lines starting with #?), and a first match logic (these files are users.conf, levels.conf, bans.dat, banLists.conf and commands.conf).

I would like to avoid repeating the same info in each file (for example the fact that commas can be used as logic OR in input fields, the fact that lines starting with #? are templates (re)definitions...). What I wanted to do initially was writing all these info in an operating guide and refer to it in the config files header. The config file headers would then only contain their specific characteristics (input fields names etc.). Unfortunately I never took the time required to write this guide so this documentation is lacking, but in my opinion this general info regarding SPADS config files really belongs in a document like that to avoid duplication of information...

However, following this logic, what is clearly missing in the commands.conf file is the list of allowed sections attributes, which can be used to override global SPADS settings for specific commands. You listed voteTime and majorityVoteMargin in your PR but there are two more: awayVoteDelay and minVoteParticipation. I will update the header to add this information, thanks again!

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.

2 participants