spack/lib/spack
Todd Gamblin 6104c31556 bugfix: be careful about GITHUB_BASE_REF in spack style
`spack style` previously used a Travis CI variable to figure out
what the base branch of a PR was, and this was apparently also set
on `develop`.  We switched to `GITHUB_BASE_REF` to support GitHub
Actions, but it looks like this is set to `""` in pushes to develop,
so `spack style` breaks there.

This PR does two things:

- [x] Remove `GITHUB_BASE_REF` knowledge from `spack style` entirely

- [x] Handle `GITHUB_BASE_REF` in style scripts instead, and explicitly
      pass the base ref if it is present, but don't otherwise.

This makes `spack style` *not* dependent on the environment and fixes
handling of the base branch in the right place.
2021-07-27 17:57:17 -07:00
..
docs Documentation does not build with Sphinx 4.1.2 2021-07-26 13:46:27 -07:00
env debug: initial global debug flag support (#24285) 2021-07-09 21:22:26 -04:00
external archspec: added support for arm compiler on graviton2 (#24904) 2021-07-15 13:27:13 +00:00
llnl API Docs: fix broken reference targets 2021-07-16 08:30:56 -07:00
spack bugfix: be careful about GITHUB_BASE_REF in spack style 2021-07-27 17:57:17 -07:00