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.

Sign up for GitHub

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

Jump to bottom

Detect unnecessary hrl inclusions #11

Open
elbrujohalcon opened this issue Dec 14, 2020 · 1 comment
Open

Detect unnecessary hrl inclusions #11

elbrujohalcon opened this issue Dec 14, 2020 · 1 comment
Labels
rule A new thing to kill with fire
Milestone
Eventually 🙄

Comments

@elbrujohalcon
Copy link
Collaborator

Find -include attributes that are not necessary, since the module doesn't use anything from the file.

⚠️ Hank has to be careful because some times those files include not just macros and records, but also attributes (e.g. -compile(export_all).).

ℹ️ There is something similar to this being developed as part of erlang-ls
@elbrujohalcon elbrujohalcon added the rule A new thing to kill with fire label Dec 14, 2020
@elbrujohalcon elbrujohalcon modified the milestones: 0.1.0, 0.2.0 Dec 14, 2020
@elbrujohalcon elbrujohalcon modified the milestones: 0.2.0, 1.0.0 Jan 7, 2021
@elbrujohalcon elbrujohalcon modified the milestones: 1.0.0, Eventually 🙄 May 12, 2021
@paulo-ferraz-oliveira
Copy link
Contributor

+1

Ran into this today.

Seems developers are keen on copying files upon files without taking care of clearing the headers 🤣

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
rule A new thing to kill with fire
Projects
None yet
Milestone
Eventually 🙄
Development

No branches or pull requests
2 participants
@elbrujohalcon @paulo-ferraz-oliveira