Skip to content
This repository has been archived by the owner on Apr 16, 2024. It is now read-only.

feat: Refactor config handling #108

Merged
merged 1 commit into from
Sep 2, 2021

Conversation

jlaw
Copy link
Contributor

@jlaw jlaw commented Aug 30, 2021

  • Rename doomrc.lua and plugins.lua to doom_modules.lua and doom_userplugins.lua, respectively.
  • Rename doom.core.config.doomrc to doom.core.config.modules`.
  • Change config files to modules to utilize requires mechanism. Add property M.source to hold load path of file.
  • Alphabetize doom_modules.lua and sync defaults with doom.core.config.modules.
  • Update help file.
  • Add doom.core.config.userplugins to be consistent.

@jlaw jlaw marked this pull request as ready for review August 30, 2021 06:29
- Rename `doomrc.lua` and `plugins.lua` to `doom_modules.lua` and `doom_userplugins.lua`, respectively.
- Rename `doom.core.config.doomrc` to doom.core.config.modules`.
- Change config files to modules to utilize `requires` mechanism. Add property `M.source` to hold load path of file.
- Alphabetize `doom_modules.lua` and sync defaults with `doom.core.config.modules`.
- Update help file.
- Add `doom.core.config.userplugins` to be consistent.
Copy link
Collaborator

@NTBBloodbath NTBBloodbath left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey, I'm pretty sorry for the late reply, my internet connection is pretty much of garbage right now lmao

The changes looks really good IMO, I'll be very glad to merge them. Pretty good job man, thanks for doing this!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants