General cleanup + refactor

core/autoload/popups.el

@@ -16,6 +16,7 @@ current window if omitted."
 ;;;###autoload
 (defun doom-popup-buffer (buffer &rest plist)
   "Display BUFFER in a shackle popup. See `shackle-rules' for possible rules."
+  (declare (indent defun))
   (unless (bufferp buffer)
     (error "%s is not a valid buffer" buffer))
   (when (and plist (not (plist-member plist :align)))

core/core-packages.el

@@ -144,7 +144,7 @@ to speed up startup."
 
     (require 'quelpa)
     (require 'use-package)
-    ;; Remove package management keywords, I'll deal with that myself
+    ;; Strip package management from use-package; DOOM handles this its own way
     (mapc (lambda (keyword) (setq use-package-keywords (delq keyword use-package-keywords)))
           '(:ensure :pin))
 

modules/app/rss/config.el

@@ -56,6 +56,6 @@
   :after elfeed
   :config
   (setq rmh-elfeed-org-files
-        (mapcar (lambda (file) (expand-file-name file +rss-org-dir))
+        (let ((default-directory +rss-org-dir))
-                +rss-elfeed-files))
+          (mapcar 'expand-file-name +rss-elfeed-files)))
   (elfeed-org))

modules/feature/workspaces/autoload/workspaces.el

@@ -229,7 +229,7 @@ workspace to delete."
         current-name))))
   (condition-case ex
       (if (+workspace-delete name)
-          (+workspace-message (format "Deleted %s workspace" name) 'success)
+          (+workspace-message (format "Deleted '%s' workspace" name) 'success)
         (error "Couldn't delete %s workspace" name))
     ('error (+workspace-error (cadr ex) t))))