Commit Graph

83 Commits

Author SHA1 Message Date
Sajith Sasidharan
6d1b95b965 GitHub Actions: add more parenthesis
https://github.community/t/and-operator-in-if-condition/154825
suggests that adding more parens might work.
2021-04-27 16:21:22 -04:00
Sajith Sasidharan
3722b8f628 GitHub Actions: quote architecture
Docs for actions/setup-python seem to do that, although
inconsistently.
2021-04-27 15:52:28 -04:00
Sajith Sasidharan
73d3295a6a GitHub Actions: use 64-bit Python 3.6 to test on Windows 2021-04-27 15:49:25 -04:00
Sajith Sasidharan
65398a2d63 GitHub Actions: update note about Windows again 2021-04-26 17:06:34 -04:00
Sajith Sasidharan
1531bea63f GitHub Actions: update note about Windows
[skip ci]
2021-04-26 16:58:24 -04:00
Sajith Sasidharan
e46e4409c2 GitHub Actions: really use x86 Python on Windows
In a prior commit, I mistakenly used "architecture: x64" instead of
"architecture: x86", and tests actually passed.

That was surprising, because netifaces do not have amd64_win will on
PyPI.  But mystery was solved when itamarst pointed out that netifaces
wheel (that we previously built) is present on pip cache.

But pip cache might be purged one day, and tests will fail again that
day.  We can't have that, so we will try to stick with x86 for now.
2021-04-26 16:39:23 -04:00
Sajith Sasidharan
68603fc015 GitHub Actions: just use x86 Python to test on Windows
I can't figure out the correct GitHub Actions magic incantation^w^w
expression syntax that is needed to isolate (Windows && Python 2.7),
so let's just run also run x86 Python 3.6 on Windows.
2021-04-26 13:41:17 -04:00
Sajith Sasidharan
dec97b3aa1 GitHub Actions: when in doubt, drop some curly braces 2021-04-26 13:35:37 -04:00
Sajith Sasidharan
ed82119f32 GitHub Actions: when in doubt, throw in more parens 2021-04-26 13:14:28 -04:00
Sajith Sasidharan
106976e8cc GitHub Actions: use expression syntax
Per GitHub documentation:

  When you use expressions in an if conditional, you may omit the
  expression syntax (${{ }}) because GitHub automatically evaluates
  the if conditional as an expression, unless the expression contains
  any operators. If the expression contains any operators, the
  expression must be contained within ${{ }} to explicitly mark it for
  evaluation.

https://docs.github.com/en/actions/reference/context-and-expression-syntax-for-github-actions
2021-04-26 12:17:25 -04:00
Sajith Sasidharan
196ce5103d GitHub Actions: test with 32-bit Python 2.7 on Windows 2021-04-26 12:00:11 -04:00
Sajith Sasidharan
b707a6ca7b GitHub Actions: do not install vcpython27
Microsoft seems to have pulled the compiler download.
2021-04-26 11:36:51 -04:00
Sajith Sasidharan
5f7c6e4552 Remove Windows exclusion 2021-04-05 18:34:48 -04:00
Sajith Sasidharan
65fd5a4912 Add a note about adding Windows to GitHub Actions test matrix 2021-04-05 14:52:31 -04:00
Sajith Sasidharan
2a62086323 Turn down coveralls verbosity 2021-03-31 19:16:58 -04:00
Sajith Sasidharan
d17f3d36c2 Run coveralls verbosely 2021-03-31 17:57:44 -04:00
Sajith Sasidharan
2e67e07081 Just don't run coverage tests with Python 3.6 + Windows
Another test matrix setup I tried is this:

  jobs:
    coverage:
        matrix:
          os:
            - macos-latest
            - windows-latest
            - ubuntu-latest
          python-version:
            - 2.7
          include:
            - python-version: 3.6
              os:
                - macos-latest
                - ubuntu-latest
            - python-version: 3.6
              os: windows-latest
              experimental: true

But that failed on Python 3.6 + macOS with a simple error message, and
no further explanation: "This check failed".  Huh?

Might simply exclude Windows altogether, because that approach sort of
worked in another experiment.
2021-03-31 16:42:39 -04:00
Sajith Sasidharan
1351a62ac4 Expect coverage tests to fail on Python 3.6 + Windows 2021-03-31 16:13:41 -04:00
Sajith Sasidharan
dd3b95a0bd Add ubuntu-latest to GitHub Actions matrix 2021-03-31 15:55:05 -04:00
Sajith Sasidharan
aebbc52f06 Add Python 3.6 to GitHub Actions test matrix
Let us send more coverage reports to coveralls.io
2021-03-31 15:55:05 -04:00
Sajith Sasidharan
88e3005abb Quote some YAML strings
Following PR review feedback.

Some parts of GitHub Actions configuration follows convention in GA's
documentation, in which YAML strings are not quoted, but that probably
is not a good idea.  We also don't want to change all the strings in
this unrelated set of changes.  So we will quote strings as we go, in
the blocks we touch.
2021-03-31 11:53:02 -04:00
Sajith Sasidharan
4e4faee5fd Update notes about coveralls versions 2021-03-27 10:21:34 -04:00
Sajith Sasidharan
626eba53b8 Run coveralls --finish to indicate completion 2021-03-27 10:17:22 -04:00
Sajith Sasidharan
32b2e93302 Use newer version of coveralls for reporting 2021-03-27 10:03:12 -04:00
Itamar Turner-Trauring
38da89a7b5 Try to unbreak Hypothesis in CI. 2021-02-23 09:52:30 -05:00
Jean-Paul Calderone
7864330562 Merge remote-tracking branch 'origin/master' into 3385.coveralls-for-coverage-reports 2021-02-12 12:58:49 -05:00
Sajith Sasidharan
ed92202762 Updates comments about GitHub cache action 2021-01-18 17:55:07 -05:00
Sajith Sasidharan
9e4ea0c491 Use fetch-depth of 0 with GitHub Actions
Using a fetch-depth of 0 should have the same effect as as `git fetch
--prune --unshallow` after doing a shallow checkout.
2021-01-18 17:54:17 -05:00
Sajith Sasidharan
99cca0ea8e No need of upgrading pip on GitHub Actions 2021-01-18 13:38:09 -05:00
Sajith Sasidharan
adbe23fe7a Add a note about pip version on GitHub Actions 2021-01-18 13:34:25 -05:00
Sajith Sasidharan
2a1a5cb0a5 GitHub Actions have sufficiently recent pip
At the time of writing this commit message, GitHub Actions offers pip
v20.3.3 for both ubuntu-latest and windows-latest, and pip v20.3.1 for
macos-latest.  Those are sufficiently recent pip versions that have
"cache dir" sub-command.
2021-01-18 13:32:53 -05:00
Sajith Sasidharan
1f1a30095e Get pip version for all three GitHub Actions OSes 2021-01-18 13:22:04 -05:00
Sajith Sasidharan
8bf068f991 What's the pip version on GitHub Actions?
There's no need of upgrading pip if GA offers a sufficiently new pip.
2021-01-18 13:19:33 -05:00
Sajith Sasidharan
f731159cd7 Install Python packages after setting up pip cache 2021-01-18 13:13:32 -05:00
Sajith Sasidharan
573ab8768b Re-title "use pip cache" step in GitHub Actions 2021-01-18 11:08:56 -05:00
Sajith Sasidharan
27a122088c Use pip cache on GitHub Actions
Using the method outlined in https://github.com/actions/cache/blob/main/examples.md#using-pip-to-get-cache-location
2021-01-18 11:02:55 -05:00
Sajith Sasidharan
fa1a8e8371 Upgrade pip used in GitHub Actions
From pip 20.1+ onward, "pip cache dir" can be used to find location of
pip cache, and this is useful across all three major OSes supported by
GitHub Actions.
2021-01-18 10:58:12 -05:00
Jean-Paul Calderone
e72c93a982 explain why we're not using the github action here 2021-01-08 08:52:38 -05:00
Jean-Paul Calderone
5614c4c3f4 improve this comment marginally
just looking for an excuse to trigger another build and see if inviting
"coveralls" to be a collaborator on LeastAuthority/tahoe-lafs fixes the status
reporting issue.
2021-01-08 08:27:00 -05:00
Jean-Paul Calderone
709823e562 most of those words proved irrelevant 2021-01-07 18:24:57 -05:00
Jean-Paul Calderone
3b8df95e3e Try constructing build_num differently for push 2021-01-06 21:09:33 -05:00
Jean-Paul Calderone
e382ef8a89 Clean up the explanation, link to some more/better stuff 2021-01-06 20:27:41 -05:00
Jean-Paul Calderone
f5ba293f79 Ideally this is no longer necessary 2021-01-06 20:27:17 -05:00
Jean-Paul Calderone
e3a6f43dc9 less shell wankery 2021-01-06 16:43:11 -05:00
Jean-Paul Calderone
59e385c00f apparently it doesn't have remotes in there 2021-01-06 16:24:53 -05:00
Jean-Paul Calderone
d515887ba1 This is probably faster and may actually work
`git fetch --prune --unshallow` doesn't seem to get
refs/remotes/pull/<PR>/merge but that's okay because HEAD is already set to that
2021-01-06 16:16:36 -05:00
Jean-Paul Calderone
89c54af01d Guess we need the rest of the repo too, surprise. 2021-01-06 16:05:35 -05:00
Jean-Paul Calderone
fac12210cb Can't do that Git stuff without a checkout
Maybe *that* is why `coveralls --finish` fails?
2021-01-06 15:58:48 -05:00
Jean-Paul Calderone
52c42b5118 dump this info, who knows how many more rounds this will take 2021-01-06 15:48:20 -05:00
Jean-Paul Calderone
24a531474d So much for my ability to read and understand a Python program
coveralls complained:

{"error":"No build matching CI build number 467026020 found"}

So try constructing a build_num that looks like the value we observed from
`coveralls` output when it was submitting coverage data.
2021-01-06 15:38:12 -05:00