Restore the old behavior when 'nginx-prometheus-exporter' was the default rule for make

[dominikwinter]

Proposed changes

This pull request resolves #620.

In the Makefile, the help and nginx-prometheus-exporter targets have been swapped, now making nginx-prometheus-exporter the default target again. This adjusts the default build command ensuring consistency with the instructions in README.md.

Please provide feedback if further changes are needed.

Checklist

Before creating a PR, run through this checklist and mark each as complete.

• I have read the CONTRIBUTING guide
• I have proven my fix is effective or that my feature works
• I have checked that all unit tests pass after adding my changes
• I have ensured the README is up to date
• I have rebased my branch onto main
• I will ensure my PR is targeting the main branch and pulling from my branch on my own fork

[lucacome]

Hi @dominikwinter

thanks for the PR! I think it would be better to update the README to use make nginx-prometheus-exporter and leave an empty command to show the help...any particular reason why you want make to build the binary?

[dominikwinter]

Hi @lucacome,

The ticket was relatively clear to me as it is a notable convention in the open source community. Many user are accustomed to the defaults like make and make install.

On the other hand, I understand clear targets like make nginx-prometheus-exporter can help newcomers.

But what's best - keep to the norm or make it as clear as possible? :)

Let me know what you think.

[lucacome]

Makes sense to me, left a comment.