Stop bots from running codecov upload step in CI

[davidorme]

Description

This is just adding conditionals on the execution of the codecov upload step in CI to stop bots running that step, because they don't seem (or at least codecov doesn't) to have access to the required secrets, unlike PRs from actual repo members.

Fixes #589

Type of change

• New feature (non-breaking change which adds functionality)
• Optimization (back-end change that speeds up the code)
• Bug fix (non-breaking change which fixes an issue)

Key checklist

• Make sure you've run the pre-commit checks: $ pre-commit run -a
• All tests pass: $ poetry run pytest

Further checks

• Code is commented, particularly in hard-to-understand areas
• Tests added that prove fix is effective or that feature works
• Relevant documentation reviewed and updated

[jacobcook1995]

LGTM!

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.09%. Comparing base (bb1211c) to head (4f501b6).
⚠️ Report is 3224 commits behind head on develop.

Additional details and impacted files

@@           Coverage Diff            @@
##           develop     #590   +/-   ##
========================================
  Coverage    95.09%   95.09%           
========================================
  Files           74       74           
  Lines         4302     4302           
========================================
  Hits          4091     4091           
  Misses         211      211           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[davidorme]

Ha! To be honest I was expecting:

1. the YAML syntax to fail or
2. the codecov job not to run here, where it should.

[davidorme]

Lets see if the rebased dependabot merges run!