aboutsummaryrefslogtreecommitdiff
path: root/.github/workflows/ci.yml
Commit message (Collapse)AuthorAge
...
* ci(uncrustify): show diff on failure #18780dundargoc2022-05-28
| | | | This will make it possible to see what needs to be fixed without having uncrustify installed locally.
* ci: bump github actions versions #18616dundargoc2022-05-20
|
* ci: lint with uncrustify #18563dundargoc2022-05-20
| | | | | | | This lint job will ensure that the C codebase is properly formatted at all times. This helps eliminate most of clint.py. To save CI time, it's faster to manually compile uncrustify and cache the binary instead of using homebrew (the apt-get package is too old).
* ci: remove mingw job #18580dundargoc2022-05-15
| | | | | | | | | | | | | | | | | | | | Unnecessary CI builds increase the change of spurious failures, which are costly noise. Of course, we should fix all legitimate bugs, but we also cannot micro-manage every platform, so there needs to be a clear motivation for the CI builds that we maintain. Reasons against maintaining a mingw CI job: 1. The windows mingw build is slow. 2. Failures: - https://github.com/neovim/neovim/issues/18494 - https://github.com/neovim/neovim/issues/18495 3. The mingw artifact is 10x bigger than the windows MSVC artifact: https://github.com/neovim/neovim/issues/10560 4. Our releases publish the MSVC (not mingw) artifact for Windows users: https://github.com/neovim/neovim/releases 5. Non-MSVCRT has limitations documented by libuv: http://docs.libuv.org/en/v1.x/process.html > On Windows file descriptors greater than 2 are available to the child process only if the child processes uses the MSVCRT runtime. Closes https://github.com/neovim/neovim/issues/18551
* ci: enable tests on drafts as well #18566dundargoc2022-05-15
| | | | | | | | | | | | | | | | | | | previous: https://github.com/neovim/neovim/pull/14123 CI tests were disabled on drafts #18566 to manage the large number of incoming jobs. While this did help, it had the drawback of making the purpose of the ready-for-review a bit fuzzier. It went from a clear "my PR is ready" signal to maintainers to somewhere between "my PR is ready but I need the tests to confirm" to "please don't merge yet, I just need to see the test results". Worse is that the specific case of wanting to see the test results but not wanting it merged is that this needs to be actively conveyed to the maintainers with a [DNM] or a comment to not merge the PR yet. All of this causes weird workarounds and noises which I believe isn't necessary. The reason why I don't think this workaround is needed anymore is that our CI now aborts a job if a new job from the same pull requests is created, which makes the "10 simultaneous jobs per PR" situations that triggered this not possible.
* ci: remove remnants of gperf #18550Justin M. Keyes2022-05-12
| | | | gperf was removed in 36613b888bae7df764a26a28ca1627a2c0c2edeb yay!
* ci: lint runtime with styluaChristian Clason2022-05-09
|
* ci: add a check with -funsigned-charzeertzjq2022-05-06
|
* ci: refactor and simplify CI processDundar Göc2022-03-10
|
* ci(macos): minimize log output from brew updateDundar Göc2022-03-07
| | | | | Also remove perl from brew install to prevent a warning that states it's already installed.
* ci: document the purpose of some non-obvious jobsDundar Göc2022-03-07
|
* ci(lint): re-add building of third-partyJames McCoy2022-02-27
|
* ci: add timeout to all jobsDundar Göc2022-02-26
|
* Merge pull request #17467 from dundargoc/ci/remove-failing-windowsJames McCoy2022-02-25
|\ | | | | ci: remove failing windows CI tests
| * ci: bump Windows image to windows-2019James McCoy2022-02-20
| | | | | | | | | | | | | | | | The VS 2019 CMake generator no longer has different generator types for different architectures. Now, the architecture is specified via CMake's `-A` switch. However, this requires we also propagate `${CMAKE_GENERATOR_PLATFORM}` to the bundled deps, so they build for the same architecture as Nvim.
* | ci: skip ci workflow on only documentation changesDundar Göc2022-02-21
| |
* | Merge pull request #17469 from jamessan/use-sysdeps-for-lint-ciJames McCoy2022-02-20
|\ \ | | | | | | Use system dependencies for lint CI
| * | ci(lint): explicity build nvimJames McCoy2022-02-19
| | | | | | | | | | | | | | | | | | “make clint-full” bypasses the normal mechanisms used to communicate build flags in the CI jobs, so explicitly build nvim before running the lint jobs.
| * | ci(lint): install deps via apt instead of building from third-partyJames McCoy2022-02-19
| |/
* / ci: remove success check since it's the defaultDundar Göc2022-02-20
|/
* Merge pull request #17404 from dundargoc/ci/put-each-test-in-separate-stepJames McCoy2022-02-18
|\ | | | | ci: run each test suite in a separate github step
| * ci: run each test suite in a separate github stepDundar Göc2022-02-18
| | | | | | | | | | This should help combat some of the lagginess when looking at the CI logs in the browser.
* | ci(lint): simplify cache keyDundar Göc2022-02-18
|/
* ci(lint): remove unnecessary stepsDundar Göc2022-02-17
|
* revert: "ci: run functionaltest-lua on drafts as well"Dundar Göc2022-02-15
| | | | This reverts commit a91a5c1880dcb7718aed4e19ff2757be0948e7d5.
* chore: fix typos (#16816)dundargoc2022-01-04
| | | | | | | Co-authored-by: Sean Dewar <seandewar@users.noreply.github.com> Co-authored-by: Gregory Anders <greg@gpanders.com> Co-authored-by: Sebastian Volland <seb@baunz.net> Co-authored-by: Lewis Russell <lewis6991@gmail.com> Co-authored-by: zeertzjq <zeertzjq@outlook.com>
* ci: install flake8 with apt instead of pipDundar Göc2022-01-03
|
* ci: remove unnecessary before_install scriptDundar Göc2022-01-02
|
* Merge pull request #16853 from jamessan/clint-master-onlyJames McCoy2022-01-02
|\ | | | | ci: run lint only on master branch
| * ci: run lint only on master branchJames McCoy2021-12-31
| | | | | | | | | | The list of known lint issues is only updated for the master branch, so other branches report lots of false positive issues.
* | Merge pull request #16840 from zeertzjq/ci-clang-13James McCoy2022-01-01
|\ \ | |/ |/| ci: bump clang version to 13 in asan and tsan
| * ci: bump clang version to 13 in asan and tsanzeertzjq2022-01-01
| |
* | ci: remove outdated brew workaround that isn't needed anymoreDundar Göc2021-12-31
|/
* ci: cancel in-progress CI if a PR is updatedJames McCoy2021-12-25
| | | | [skip ci]
* ci: remove "brew upgrade" from macos jobsDundar Göc2021-12-04
| | | | | | Packages are automatically upgraded on install. This will avoid upgrading unrelated packages, cutting the macos job time to about a half.
* ci!: remove win32 builds/releasesJames McCoy2021-11-14
|
* ci: limit CI checks to the main dev branchesJames McCoy2021-10-31
| | | | | | | | Now that we have various GH actions creating branches in the main repo, using the generic '**' pattern for the CI workflow is just wasting CI time and leading to more queued jobs. [skip ci]
* ci: run functionaltest-lua on drafts as wellDundar Göc2021-10-28
|
* build: update cmake min version to 3.10 #16065Jakub Łuczyński2021-10-19
| | | | | | * build(cmake): update cmake min version to 3.10 * ci: test cmake minimum required version * build(cmake): remove some legacy includes * Since version 3.5 cmake_parse_arguments is implemented natively.
* ci: use runner, instead of os, for job name and cache keyJames McCoy2021-09-24
|
* ci: add macOS 11 buildJames McCoy2021-09-24
|
* fix(ci): re-run GHA for ready_for_review events (#15377)James McCoy2021-08-15
| | | | | | | | By default, the pull_request target only runs for the "opened", "synchronize", and "reopened" events. Since we've disabled GHA CI for "draft" PRs, the "ready_for_review" event also needs to be included so CI is automatically run when leaving draft status.
* fix(ci): lower timeout to 45 minutes (from default 360 mins)Björn Linse2021-07-24
|
* Revert "fix(ci): remove mongodb from brew to avoid brew upgrade error ↵Christian Clason2021-07-20
| | | | | | | (#15115)" (#15141) This reverts commit 5377b2b00aea1a0bde1b81452e6198dabe5b9796. (Fix no longer needed.)
* fix(ci): remove mongodb from brew to avoid brew upgrade error (#15115)Oliver Marriott2021-07-17
| | | | | | | | | macOS CI was failing because: - brew upgrade fails because, - mongodb-community cant upgrade because, - some symlinks are owned by ... mongodb-community... We don't use mogodb, so we can just remove it wholesale.
* ci(gha): Add functionaltest-lua checkerJames McCoy2021-06-29
|
* fix(ci): Remove gcc@8/gcc@9 homebrew workaroundJames McCoy2021-06-04
|
* ci(gha): Run ci workflow for PRs into release branchesJames McCoy2021-05-05
|
* ci(gha/ci): Use clang-12 for TSANJames McCoy2021-05-02
|
* ci(gha/win): Specify valid directory for DEPS_BUILD_DIRJames McCoy2021-04-21
|