Athanasius
c2fdd0a319
pr-checks: Try not changing options to actions/checkout@v2
2021-04-12 08:31:48 +01:00
Athanasius
34d4e72bc4
GitHub workflows: echo info about what we're git diff'ing
...
Whenever A_D does a PR the checks, for some unclear reason, also contain
'other' files causing unexpected flake8 output.
i.e. PR that changes only plugins/inara.py somehow "git diff .. | flake8"
such that flake8 complains about things in collate.py, coriolis.py,
loadout.py
2021-04-12 08:31:47 +01:00
Athanasius
30db492212
Bump to Python 3.9.4
...
I can PyCharm debug with this, unlike 3.9.3 :)
2021-04-12 08:31:47 +01:00
Athanasius
f2b89691c6
Python: Use 3.9.3
2021-04-12 08:31:46 +01:00
Athanasius
180d5b0cb6
Bump Python 3.8 -> 3.9 in {pr,push}-checks.yml GH workflows
...
Closes #893
2021-04-01 14:45:50 +01:00
Athanasius
31da53200b
Bump python version in the github windows build workflow
2021-04-01 14:45:49 +01:00
A_D
2ff06c15ab
Fix double quoted strings in workflow
2021-04-01 14:45:47 +01:00
A_D
c2faad1eb7
Removed uneeded whitespace
2021-04-01 14:45:45 +01:00
A_D
1c80578fef
Fixed backwards pip install command
2021-04-01 14:45:45 +01:00
Athanasius
c69cc77682
Added auto-annotation to both checks
...
This works based on telling github how to understand the flake8 output
# Conflicts:
# .github/workflows/push-checks.yml
2021-04-01 14:45:45 +01:00
Athanasius
7e094df6c4
Switch to Python 3.8.6 for windows builds on GitHub
2021-04-01 14:45:45 +01:00
Athanasius
6600d1a78a
GitHub Workflow: Update to Python 3.8
2021-04-01 14:45:12 +01:00
A_D
d58d831ebe
Replace python checks with jq in workflow
...
Also makes sure that pushes to main ARE checked
2021-01-05 17:20:21 +02:00
A_D
bd74aaf0aa
Add name to windows build action
...
This should make it show up with a more sane name
2020-12-08 19:36:26 +02:00
A_D
033a3bb4da
Removed appcast, only upload msi installer
2020-12-08 18:51:00 +02:00
A_D
87afa69f53
Removed py2exe setup
2020-12-08 18:37:33 +02:00
A_D
fa3335d59a
Added windows build
2020-12-08 18:37:32 +02:00
A_D
009ded50a1
Ensured PRs dont fail on doc lints
2020-09-09 23:05:59 +02:00
Athanasius
3dab0e9d91
Comment pr-check flake8, and don't --exit-zero the full check
2020-07-31 14:44:44 +01:00
Athanasius
4aa841c6e2
Will PY file check work if env.PYFILES not set at all ?
2020-07-31 13:59:52 +01:00
Athanasius
67a13d3869
Change Checkout conditional to direct context, not env vars
2020-07-31 13:56:18 +01:00
Athanasius
e087cd7f85
Comment that 'env' for another commit
2020-07-31 13:53:23 +01:00
Athanasius
ecf4732e44
Check step needs a 'uses' or 'run', so add latter with 'env' for check
2020-07-31 13:51:50 +01:00
Athanasius
937dacd7ca
Implement "same owner?" check for base/head
...
* Also updates top comment.
2020-07-31 13:47:40 +01:00
Athanasius
3972abbace
Comment out the current steps
2020-07-31 13:42:00 +01:00
Athanasius
b75e2ec2b1
Comment about the check being wrong
...
This gets the branch rolling for a PR
2020-07-31 13:36:49 +01:00
Athanasius
64e6ad439c
Try remote-add/fetch in separate step
2020-07-31 13:28:28 +01:00
Athanasius
8d7f91248d
Oops, moved the 'steps:' line too
2020-07-31 13:18:46 +01:00
Athanasius
063a83b8e8
Move 'Show github context' to just before flake8
...
$GITHUB_EVENT_PATH wasn't set in the shell with it first.
2020-07-31 13:17:56 +01:00
Athanasius
71fa1dbcf6
Comment the 'add base remote and fetch it'
2020-07-31 13:15:52 +01:00
Athanasius
3d46f23fc7
Need svn_url, not url
...
'url' is api.github.com
2020-07-31 13:07:40 +01:00
Athanasius
895376c039
Try manually adding remote and fetching to get the refs
...
Also, add 'end' to 'Show github context'
2020-07-31 13:05:20 +01:00
Athanasius
62fe9deae7
repo.owner.login not repo.full_name
2020-07-31 12:57:44 +01:00
Athanasius
e1a5a1f811
Top-level event key needs to be pull_request
2020-07-31 12:51:07 +01:00
Athanasius
7c00ddcebf
We want github.context for the full info
2020-07-31 12:40:38 +01:00
Athanasius
2fa10b4a9e
Correct setting of BASE_REPO
2020-07-31 12:35:54 +01:00
Athanasius
c5d43a4ada
Try a whole new method of referring to 'base', and checkout 'head'
2020-07-31 12:34:12 +01:00
Athanasius
92ea8943df
Run 'Show github context' as first step
2020-07-31 12:21:32 +01:00
Athanasius
a01c36f660
Correct "-run:" to "run:"
2020-07-31 12:20:12 +01:00
Athanasius
015a50d53f
Fix small typo "-name" -> "- name"
2020-07-31 12:18:05 +01:00
Athanasius
43ae6dde61
Restore changes, draft status was to blame ?
2020-07-31 12:15:15 +01:00
Athanasius
91e07a62eb
Revert pr-checks.yml to see if it runs again
2020-07-31 12:12:53 +01:00
Athanasius
5d7612b0db
Move comment, was it stopping any steps from being defined?
2020-07-31 12:09:05 +01:00
Athanasius
9259afd344
Add display of entire github context
2020-07-31 11:58:13 +01:00
Athanasius
c18f183db9
.git would be a directory if present, fix test
2020-07-29 15:00:39 +01:00
Athanasius
59babd113b
Just use the single allowed actor for now.
...
And it's now set to "Athanasius".
2020-07-29 15:00:39 +01:00
Athanasius
7a44c28b95
Revert "Try contains(['Athanasius'], github.actor)"
...
This reverts commit 72c201c5a1 .
2020-07-29 15:00:39 +01:00
Athanasius
e22fadd5f5
Revert "Set array in earlier job using JSON"
...
This reverts commit 8b863c5efd .
2020-07-29 15:00:39 +01:00
Athanasius
51f92b3793
Revert "No nesting ${{..}}"
...
This reverts commit 9ed541f0e0 .
2020-07-29 15:00:39 +01:00
Athanasius
0736bbf97b
Revert "JSON must have double quotes"
...
This reverts commit 4db2c907df .
2020-07-29 15:00:39 +01:00