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

config: correct typo to ensure consistency between example and code #302

Merged
merged 1 commit into from
Oct 6, 2022

Conversation

GovanifY
Copy link
Contributor

There is a typo in the sample config file, reporting the pluginDir option as pluginsDir;
The code uses pluginDir instead (see:

) and isn't documented anywhere else, so it might be a nice idea to correct this typo.

@Half-Shot
Copy link
Collaborator

Signed-off-by: Gauvain 'GovanifY' Roussel-Tarbouriech <gauvain@govanify.com>
@GovanifY
Copy link
Contributor Author

GovanifY commented May 18, 2022

@Half-Shot Done! The commit was already signed-off but I didn't include a changelog in the original commit as the change seemed minor enough, it's now done.

Also, what use does changelog.d really have? Couldn't you just read off the git log and bisect if something went wrong/you wanted to see changes? I could understand a properly formatted changelog of new features or breaking changes, but I don't see the point for this specific implementation.

@Half-Shot
Copy link
Collaborator

Also, what use does changelog.d really have? Couldn't you just read off the git log and bisect if something went wrong/you wanted to see changes? I could understand a properly formatted changelog of new features or breaking changes, but I don't see the point for this specific implementation.

I see your point, but anything user facing generally has to end up in the changelog unless it's absolutely minuscule. sample configs are one of those things I like to make obvious though.

@Half-Shot Half-Shot merged commit 2ee1af5 into matrix-org:develop Oct 6, 2022
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