| Commit message (Collapse) | Author | Age |
... | |
| |
| |
| |
| |
| |
| | |
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.
|
|/ |
|
|
|
|
| |
This will prevent a failure for backported PRs as they use a different
commit message.
|
| |
|
|
|
|
|
|
|
| |
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).
|
|
|
|
|
| |
Neovim currently falls in the 100 kLOC - 500 kLOC bucket, which can scan
3x/day.
|
|
|
|
| |
The comment says it should be run at 00:10 UTC, and in cron job format
minutes come before hours.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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.
|
|
|
|
| |
gperf was removed in 36613b888bae7df764a26a28ca1627a2c0c2edeb
yay!
|
| |
|
| |
|
|
|
|
| |
Triggering by comment is not needed. Applying the label is enough to
trigger the backport action.
|
|
|
|
|
|
|
|
|
|
| |
- 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.
|
|\ |
|
| | |
|
|\ \
| |/
|/| |
ci(doc): improve missing docs workflow
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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]
|
|\ \
| |/
|/| |
ci: automatic breaking change label
|
| |
| |
| |
| |
| | |
When the PR title contains the breaking change format apply the
breaking-change label.
|
|/
|
|
| |
Co-authored-by: Jordan Haine <jhaine@securitycompass.com>
|
|
|
|
| |
After #17623 this pattern is no longer necessary.
|
|
|
|
|
| |
Also error on the side of security adding an extra check on the
automatic PR step.
|
|
|
|
|
|
|
|
| |
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]
|
| |
|
|
|
|
|
| |
Also remove perl from brew install to prevent a warning that states it's
already installed.
|
| |
|
| |
|
| |
|
| |
|
|\
| |
| | |
ci: remove failing windows CI tests
|
| |
| |
| |
| |
| |
| |
| |
| | |
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: improve cpack packaging
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
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.
|
| | | |
|
|\ \ \
| | | |
| | | | |
Use system dependencies for lint CI
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
“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.
|
| | | | |
|
| | |/
| |/| |
|
|/ / |
|
|\ \ |
|
| | |
| | |
| | |
| | | |
This makes testing the workflows much smoother.
|
|\ \ \
| | | |
| | | | |
ci: run each test suite in a separate github step
|
| |/ /
| | |
| | |
| | |
| | | |
This should help combat some of the lagginess when looking at the CI
logs in the browser.
|
|/ / |
|
| | |
|
| | |
|
| |
| |
| |
| | |
This reverts commit a91a5c1880dcb7718aed4e19ff2757be0948e7d5.
|
| | |
|
| | |
|