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

The instrumentation functions should be snake_case by Elixir conventions but need to follow the MIB as camelCase #25

Open
petrn opened this issue Sep 13, 2023 · 0 comments

Comments

@petrn
Copy link

petrn commented Sep 13, 2023

The Erlang SNMP allows to map the MIB objects to instrumentation functions through the xxx_MIB.funcs like this:
{myName, {ex1, my_name, []}}.

The MIB compilation step in elixir-snmp should map the camelCase object names to snake_case automatically.
When the instrumentation function names are left in camelCase as in the MIB, mix credo reports readability issues like this:

┃ [R] ↗ Function/macro/guard names should be written in snake_case.
┃ apps/snmp_agent/lib/snmp_agent/snmp/mib/strouter.ex:43:7 #(SnmpAgent.Mib.StRouter.inputsTable)

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

No branches or pull requests

1 participant