aboutsummaryrefslogtreecommitdiff
path: root/.github/workflows
Commit message (Collapse)AuthorAge
...
| * ci(api-docs): upgrade to ubuntu 22 and remove conda dependencyDundar Goc2022-05-24
| | | | | | | | | | | | Ubuntu 22 has doxygen version 1.9.1 available in apt, which means we don't need to use conda anymore. This will somewhat simplify the workflow.
* | ci(api-docs): add comment explaining why full clone is requiredDundar Goc2022-05-24
|/
* ci(commitlint): only trigger on PRs to master #18679dundargoc2022-05-22
| | | | This will prevent a failure for backported PRs as they use a different commit message.
* 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(coverity): run every day instead of once/weekJames McCoy2022-05-20
| | | | | Neovim currently falls in the 100 kLOC - 500 kLOC bucket, which can scan 3x/day.
* ci(coverity): correct cron job time #18590zeertzjq2022-05-16
| | | | The comment says it should be run at 00:10 UTC, and in cron job format minutes come before hours.
* 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(backport): remove triggering by comment (#18210)zeertzjq2022-04-23
| | | | Triggering by comment is not needed. Applying the label is enough to trigger the backport action.
* refactor(packaging): Windows: improve MSI, remove NSIS #18069Henry Fraser2022-04-11
| | | | | | | | | | - Removed NSIS installer. - Prevents undefined behaviour when two installations are performed to the same directory (NSIS + MSI). - Reduced cost of maintaining two installers that do the same thing. - Chose Wix MSI due to its better integration with Windows. - Added Wix patch file to add neovim binaries to the system path during installation. - Replaced neovim installer icons with better looking versions. - Renamed neovim installer icons from logo.ico -> neovim.ico for all icons to better reflect contents.
* Merge pull request #17888 from dundargoc/ci/remove-reviewers-when-draftingJames McCoy2022-03-31
|\
| * ci: remove reviewers when drafting or closing a PRDundar Göc2022-03-27
| |
* | Merge pull request #17873 from muniter/jl-apidocs-improvementesJames McCoy2022-03-27
|\ \ | |/ |/| ci(doc): improve missing docs workflow
| * ci(doc): improve missing docs workflowJavier López2022-03-26
| | | | | | | | | | | | | | | | | | | | 1. Add new pattern `runtime/doc/**`. This is a common case were the contributor modifies only the help file but the doc gen would discard their changes. 2. Add to the output what the changes after running doc gen would be. [skip ci]
* | Merge pull request #17857 from muniter/jl-breaking-change-labelJames McCoy2022-03-26
|\ \ | |/ |/| ci: automatic breaking change label
| * ci: automatic breaking change labelJavier López2022-03-26
| | | | | | | | | | When the PR title contains the breaking change format apply the breaking-change label.
* | chore: fix typos (#17755)dundargoc2022-03-25
|/ | | | Co-authored-by: Jordan Haine <jhaine@securitycompass.com>
* ci: remove src/nvim/**.lua from docgen patternzeertzjq2022-03-24
| | | | After #17623 this pattern is no longer necessary.
* fix(ci): provide necessary permissions for calling workflowJavier López2022-03-21
| | | | | Also error on the side of security adding an extra check on the automatic PR step.
* ci(docs): add a check for PR's to commit their doc changesJavier López2022-03-18
| | | | | | | | Repurpose the api-docs workflow to also run in all PR's but work only as a check, if the changes in the PR introduce doc changes that are not committed fail. [skip ci]
* 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(lint): use bundled luv until system packages are updatedJames 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.
* | Merge pull request #17390 from RenFraser/feature/packagingJames McCoy2022-02-24
|\ \ | | | | | | ci: improve cpack packaging
| * | ci: improved cpack packagingHenry Fraser2022-02-24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Addresses: #12571 - Added the following installers through CMake files: - Windows NSIS. - Windows MSI. - Windows zip. - MacOs tarball. - Linux tarball. - Linux Deb package. - Tweaked pipeline CPack commands to build using new CMakeLists.txt configuration file. - Added icons and relevant packaging files. - Updated notes.md to reflect new installation instructions. This isn't meant to be the perfect solution, it's simply a first pass at using a simple packaging system to build Windows installers. A Debian package has also been added since it's very easy but other packages have been left out due to limiting the scope. Hopefully we can build further upon this and improve it over time with code signing, better icons and more user-friendly installation graphics and so on.
* | | 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): build against system depsJames McCoy2022-02-19
| | | |
| * | | 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 #17445 from dundargoc/ci/review/use-checkoutJames McCoy2022-02-18
|\ \
| * | ci: download reviews.js through checkout instead of wgetDundar Göc2022-02-17
| | | | | | | | | | | | This makes testing the workflows much smoother.
* | | 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
| |
* | ci(reviews): move reviews.js to .github/scriptsDundar Göc2022-02-17
| |
* | revert: "ci: run functionaltest-lua on drafts as well"Dundar Göc2022-02-15
| | | | | | | | This reverts commit a91a5c1880dcb7718aed4e19ff2757be0948e7d5.
* | ci(labeler): disable sync that interferes with other jobsDundar Göc2022-02-14
| |
* | ci: use a separate script for request review workflowzeertzjq2022-02-14
| |