include arduino_secrets when needed

[fstasi]

No description provided.

[per1234]

The only possible issue I found after a lot of testing of this change is that if you write something like this in the local sketch:

#ifndef SECRET_SOMETHING
#include "arduino_secrets.h"
#endif

(you might do this to allow the normal secret definition to be overridden via a -D compiler flag for automation or testing)

Then after a push and pull, it turns into this:

#include "arduino_secrets.h"
#ifndef SECRET_SOMETHING

#endif

---

It's reasonably common to use whitespace after the # of a preprocessor directive to provide indentation:

#ifndef SECRET_SOMETHING
#  include "arduino_secrets.h"
#endif

which turns into this after a push and pull:

#include "arduino_secrets.h"
#ifndef SECRET_SOMETHING
#  include "arduino_secrets.h"
#endif

But I think that it's unlikely for users will do something like this and also a lot of work to allow for something like this.