ED/EDMarketConnector
1
0
Fork 0
mirror of https://github.com/EDCD/EDMarketConnector.git synced 2025-04-16 17:12:21 +03:00
Code Issues Projects Releases Wiki Activity
1,833 Commits 16 Branches 276 Tags
Commit Graph

40 Commits

Author SHA1 Message Date
Athanasius
6c9bffcaa0 Don't run flake8_annotate if no access 
This is using github.actor and checking who it is.  If they're not
someone who should have access then use a run with 'exit 1' to fail the
check job.

Then flake8_annotate job needs check_access to have succeeded.

This *will* flag the workflow action as failed if no access, hopefully
it will be obvious why.

Using 'AthanasiusTEST' so it fails for me on this commit/push.
 2020-07-29 15:00:39 +01:00
Athanasius
dc58d0f3af S I N G L E Q U O T E S 2020-07-29 15:00:39 +01:00
Athanasius
09e39a47ef Attempt to limit flake8 annotations to Athanasius 2020-07-29 15:00:39 +01:00
Athanasius
24cf445254 Gurantee exit 0 AND fix the echo 
* GH Workflows run under "/bin/bash -e", which means if anything in it
 fails it all exits immediately.  So append " || true" to the egrep.
* The echo to magically set the env var back in the workflow needs to
 use "" not '' else the in-shell variable won't interpolate.
 2020-07-29 15:00:39 +01:00
Athanasius
40dc913367 *.py check should always exit 0 2020-07-29 15:00:39 +01:00
Athanasius
a548d383a3 Single quotes then ? 2020-07-29 15:00:39 +01:00
Athanasius
a93cfb5ed8 Try ENV var for passing presence of *.py files 
This uses <https://docs.github.com/en/actions/reference/workflow-commands-for-github-actions#using-workflow-commands-to-access-toolkit-functions>
to set PYFILES env var globally in the job, so should be testable in
the following steps.
 2020-07-29 15:00:39 +01:00
Athanasius
792583bfe1 *.py exit status was inverted 
That'll teach me for testing with a commit only affecting *.yml files.
 2020-07-29 15:00:39 +01:00
Athanasius
9c413a8ba9 Remove extraneous 'needs:' 2020-07-29 15:00:39 +01:00
Athanasius
dff1e3bce2 Do *.py check as step in flake8_annotate 
And make both the 'Annotate with Flake8' and 'Set up Python 3.7' steps
dependent on it.
 2020-07-29 15:00:39 +01:00
Athanasius
3ae0b83ac2 Only run flake8 annotation if there are *.py files in diff 2020-07-29 15:00:39 +01:00
Athanasius
629a3461f6 Final clean ups. 
It works!  So, yes, need refs/remotes/origin/ prepended to the base_ref,
which will always be a branch for a PR.

* Remove debug lines (set -x, etc).
* No need for pytest here as yet.
* Fall back to requirements.txt if no requirements-dev.txt.
 2020-07-29 15:00:39 +01:00
Athanasius
648c1af339 See if prepending "refs/remotes/origin/" helps 2020-07-29 15:00:39 +01:00
Athanasius
f2fa489819 Attempt "git diff <branch> --" 
As in "git diff [<options>] [<commit>] [--] [<path>...]", in case
it's an ambiguity rather than not knowing what 'develop' is.
 2020-07-29 15:00:39 +01:00
Athanasius
4a965bb7c9 Extra git commands for diagnosis 2020-07-29 15:00:39 +01:00
Athanasius
3f97588ab6 Try "fetch-depth: 0" on the extra checkouts 2020-07-29 15:00:39 +01:00
Athanasius
18ff1c0e3f Add 'set -x' to get feedback on all bash commands run 2020-07-29 15:00:39 +01:00
Athanasius
d36e213ae2 Correct "-name" -> "- name" 2020-07-29 15:00:39 +01:00
Athanasius
3ad8bd6cc8 pr-checks: Changed to pull_request and fleshed out 
* Name all steps
* After checkout@v2 fetch: 1) checkout base_ref, 2) checkout head_ref
* Set and use BASE_REF env var
 2020-07-29 15:00:39 +01:00
Athanasius
56e33d20d6 Copy push-checks.yml to pr-checks.yml 2020-07-29 15:00:39 +01:00
Athanasius
755e401df8 Revert "Change the push-checks back into also PRs." 
This reverts commit 4da1d9df4ed14909574a69c70d7d0dd3d228e529.
 2020-07-29 15:00:39 +01:00
Athanasius
4da1d9df4e Change the push-checks back into also PRs. 
Renamed to suit.
 2020-07-28 13:45:55 +01:00
Athanasius
6ba1f77352 Remove greet-everyone workflow, we've gotten things working now 2020-07-26 23:04:37 +01:00
Athanasius
df3f70b7ed Rename to push-checks.yml, don't do PRs 
Also commented better at the top
 2020-07-26 23:03:23 +01:00
Athanasius
6513db3aca Don't force --show-source in .flake8 ! 
This was the cause of our flake8-your-pr issue!  It meant that as well
as `--format json` giving JSON output it also got polluted with the raw
string of the source line as well.
 2020-07-26 23:03:23 +01:00
Athanasius
d65176aed8 Add flake8-json to requirements-dev.txt 2020-07-26 23:03:23 +01:00
Athanasius
8dbb275dca Specify GITHUB_TOKEN for flake8-your-pr via env 2020-07-26 23:03:23 +01:00
Athanasius
63d44bf8d7 Attempt to run "tayfun/flake8-your-pr@master" in PRs. 2020-07-26 23:03:22 +01:00
Athanasius
333ba24db4 Rename to 'what it runs on' prefix 2020-07-26 23:03:06 +01:00
Athanasius
e461fa62f6 The configuration must be inside a 'with:' 2020-07-26 23:01:10 +01:00
Athanasius
4a3c74ce5b Use fetch-depth 0 for 'all' on checkout@v2 2020-07-26 23:01:10 +01:00
Athanasius
f41eb4fb92 Try A_D's "diff only" flake8 checking 2020-07-26 23:01:10 +01:00
Athanasius
838e55710d Uncomment flake8 part of build in workflow 2020-07-26 23:01:10 +01:00
Athanasius
a76aae9737 Remove pytest and comment out flake8 for now. 
Narrowing down what's working and what isn't in this.
 2020-07-26 23:01:10 +01:00
Athanasius
607770f7d4 Add greet-example to github workflows 2020-07-26 20:23:02 +01:00
Athanasius
04fa3fae2c
Update python-app.yml 
Change to ubuntu-18.04
 2020-07-26 18:57:52 +01:00
Athanasius
f7968bd0c8
Update python-app.yml 
Change name to be more obvious when viewing in Actions.
 2020-07-26 18:49:10 +01:00
Athanasius
525baf8b03
Create python-app.yml 
Add a GitHub Action to check requirements-dev.txt, flake8, and pytest.
 2020-07-26 18:27:24 +01:00
A_D
d41dc38162 Create Enhancement issue template 
closes #599
 2020-07-24 13:45:17 +01:00
Athanasius
085333c80b Update issue templates 2020-07-05 19:46:02 +01:00