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

Make it more friendly with use-package #153

Merged
merged 1 commit into from
May 14, 2021
Merged

Conversation

cowboyd
Copy link
Member

@cowboyd cowboyd commented May 14, 2021

It's redundant to have to declare the use package and then also require it.

(use-package frontside-javascript :config (frontside-javascript))

Instead, we'd like the fact that you're using use-package explicitly to declare your intent to actually use the package. In other words, be able to say:

(use-package frontside-javascript)

This defines, via autoload, a hook that will be run automatically by use-package when frontside-javascript loads. This saves us the typing.

Note that the use-package-inject-hooks variable must be non-nil.

It's redundant to have to declare the use package and then also
require it.

```elisp
(use-package frontside-javascript :config (frontside-javascript))
```

Instead, we'd like the fact that you're using `use-package` explicitly
to declare your intent to actually use the package. In otherwords, be
able to say:

```elisp
(use-package frontside-javascrip)
```

This defines, via autoload, a hook that will be run automatically by
`use-package` when frontside-javascript loads. This saves us the
typing.

Note that the [`use-package-inject-hooks`][1] variable must be
non-nil.

[1]: https://github.com/jwiegley/use-package/blob/master/use-package-core.el#L257-L281
@cowboyd cowboyd merged commit f3e5c61 into release May 14, 2021
@cowboyd cowboyd deleted the use-package-friendly branch May 14, 2021 00:28
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.

1 participant