[minor] Change codacy to default off

[liamhuber]

With ruff and black across the whole repo (excl. docs) by default in the next actions release, I don't see a need to keep running codacy. From what I understand it may be useful if we want to apply additional screenings for things like cryptographic or license compliance, but we're not really leveraging any of that afaik. I'm happy to keep the functionality available, but I'm inclined to make it on-request instead of always there.

Thoughts?

[XzzX]

I think a manual on-demand trigger, if that is possible with github, would be best.

[liamhuber]

I think a manual on-demand trigger, if that is possible with github, would be best.

Triggers are not something we can handle here, that's up to the downstream repos, but it is certainly possible. What's up to us then to structure the reusable workflows in a way that is useful for those repos. I think we've already got that though, since the codacy stuff is encapsulated in the tests-and-coverage.yml reusable workflow, which is in turn called by the master push-pull.yml workflow. So having it default off works fine for everyone calling

name: Push-Pull

on:
  push:
    branches: [ main ]
  pull_request:

jobs:
  pyiron:
    uses: pyiron/actions/.github/workflows/[email protected]
    secrets: inherit

And downstream repos are further already free to additionally define something like

name: Do Codacy

on: workflow_dispatch

jobs:
 codacy:
   uses: pyiron/actions/.github/workflows/[email protected]
   secrets: inherit
   with:
     do-codacy: true