You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
As mentioned in #18, Emacs is very particular about the expected structure of packages. #210 added a regression, if we can call it that in this context.
All the tests now have function stubs, which is great, but the packages now don't follow the documentation guidelines (;;; Code: as part of the package start is missing).
My suggestion would be to
add the ;;; Code: comment to the start of the package again
To make sure that the guidelines are always followed we could add checkdoc to the CI test run (#233)
edit: I realize that this might seem a bit pedantic but I think it's important to follow the conventions, so we lead people, who are learning the language, by example.
The text was updated successfully, but these errors were encountered:
As mentioned in #18, Emacs is very particular about the expected structure of packages.
#210 added a regression, if we can call it that in this context.
All the tests now have function stubs, which is great, but the packages now don't follow the documentation guidelines (
;;; Code:
as part of the package start is missing).My suggestion would be to
;;; Code:
comment to the start of the package againTo make sure that the guidelines are always followed we could add checkdoc to the CI test run (#233)
edit: I realize that this might seem a bit pedantic but I think it's important to follow the conventions, so we lead people, who are learning the language, by example.
The text was updated successfully, but these errors were encountered: