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

Elisp Support #144

Closed
ericdallo opened this issue Jun 18, 2020 · 11 comments
Closed

Elisp Support #144

ericdallo opened this issue Jun 18, 2020 · 11 comments
Labels
enhancement New feature or request O: stale 🤖 Stale issue/pr

Comments

@ericdallo
Copy link

Is your feature request related to a problem? Please describe.

super-linter does not support Emacs lisp (elisp)

Describe the solution you'd like

Use super-linter to lint Emacs lisp (elisp) code

Describe alternatives you've considered

Byte compiling and/or checkdoc with Emacs.
Example of a GH Action: https://github.com/emacs-lsp/lsp-mode/blob/master/.github/workflows/test.yml.

@ericdallo ericdallo added the enhancement New feature or request label Jun 18, 2020
@github-actions
Copy link
Contributor

This issue has been automatically marked as stale because it has not had recent activity.
It will be closed in 14 days if no further activity occurs.
Thank you for your contributions.

If you think this issue should stay open, please remove the O: stale 🤖 label or comment on the issue.

@github-actions github-actions bot added the O: stale 🤖 Stale issue/pr label Jul 19, 2020
@ericdallo
Copy link
Author

I'm still interested in that...

@github-actions github-actions bot removed the O: stale 🤖 Stale issue/pr label Jul 19, 2020
@admiralAwkbar
Copy link
Collaborator

@ericdallo if you can make a PR against the super-linter to help get this added, I would be glad to help get it across the finish line:)

@ericdallo
Copy link
Author

Thank you @admiralAwkbar, I started a draft PR here: https://github.com/github/super-linter/pull/530 but I'm stuck not being able to install emacs on the docker image correctly:
image
I also found this: cask/cask#471

@akirak
Copy link

akirak commented Aug 15, 2020

Hello, I am currently working on elinter, which can be a solution to this issue.

It is still in alpha, but I've implemented its basic features. Now I'm starting to apply it to existing projects to test its functionality. If it works, I will start working on integrating it with super-linter.

One question: elinter depends on the Nix package manager. Can I install it into the Docker container, or is there any other integration that uses it?

@ericdallo
Copy link
Author

Nice @akirak, I think you can install on the docker image like many other linters are doing that (and I think nix should be used from other linters in the future too)

@github-actions
Copy link
Contributor

This issue has been automatically marked as stale because it has not had recent activity.
It will be closed in 14 days if no further activity occurs.
Thank you for your contributions.

If you think this issue should stay open, please remove the O: stale 🤖 label or comment on the issue.

@github-actions github-actions bot added the O: stale 🤖 Stale issue/pr label Sep 15, 2020
@akirak
Copy link

akirak commented Sep 15, 2020

Sorry, I won't work on this issue soon. I don't mind if it's closed for now so that it can be reopened later. If someone else would implement it, please go ahead.

@github-actions github-actions bot removed the O: stale 🤖 Stale issue/pr label Sep 15, 2020
@akirak
Copy link

akirak commented Sep 15, 2020

In elinter-demo, I randomly pick a package from top 25% ones on MELPA every day and run lint on it, and no package has ever passed the checks. Maybe you'll want to discuss which checks are essential.

@github-actions
Copy link
Contributor

This issue has been automatically marked as stale because it has not had recent activity.
It will be closed in 14 days if no further activity occurs.
Thank you for your contributions.

If you think this issue should stay open, please remove the O: stale 🤖 label or comment on the issue.

@github-actions github-actions bot added O: stale 🤖 Stale issue/pr and removed O: stale 🤖 Stale issue/pr labels Oct 16, 2020
@super-linter super-linter deleted a comment from tarsius Oct 16, 2020
@github-actions
Copy link
Contributor

This issue has been automatically marked as stale because it has not had recent activity.
It will be closed in 14 days if no further activity occurs.
Thank you for your contributions.

If you think this issue should stay open, please remove the O: stale 🤖 label or comment on the issue.

@github-actions github-actions bot added the O: stale 🤖 Stale issue/pr label Nov 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request O: stale 🤖 Stale issue/pr
Projects
None yet
Development

No branches or pull requests

3 participants