Use devclean on elixir target for consistency of Makefile

[eiri]

Overview

This is a simple change of Makefile for consistency sake. Since dev/run overrides etc configs on each run functionally it's the same as earlier @rm command.

Testing recommendations

make elixir should pass, apart from common races.

Related Issues or Pull Requests

Checklist

• Code is written and works correctly;
• Changes are covered by tests;
• Documentation reflects the changes;

[wohali]

Please update Makefile.win at the same time!

[eiri]

@wohali Damn it! Sorry, I keep forgetting about Windows. Thank you for the remainder, will do in a moment.

[eiri]

@wohali Makefile.win updated. Sorry, I don't have a Win machine even as a VM, so can't actually test it. 😞

[dottorblaster]

If we merge this, I'm going to update #1769 to include devclean for elixir-credo too. Kudos!

[wohali]

@dottorblaster Am wondering if we shouldn't split the credo/format/black syntax checks out into a new target that check calls, something like syntax-check that does all the syntax checks at once. We can then add syntax-check-update to actually format the code for developer happy fingers pre-PR.

[wohali]

@eiri +1 good to merge

[dottorblaster]

@wohali I think we should address one issue at the time 🙂It's fine of course, but I think we need a roadmap, just like:

• We merge Black checks
• We merge Credo checks
• We refactor those check in a syntax-check goal, addressing a subsequent issue

Is that ok for you? I think we are going to cross reference too much PRs otherwise 😄

[wohali]

@dottorblaster Yes, sorry, just wanted to get this idea out there, we could also take it to dev@couchdb.apache.org where it belongs :)

[dottorblaster]

@wohali you did the right thing of course :D