14 Commits (b20cf15eef6d31f81d853d86759a68e2b9e54da8)

Author	SHA1	Message	Date
Clément Delafargue	f386ff795d	Check for external file modifications when writing (#5805) ... `:write` and other file-saving commands now check the file modification time before writing to protect against overwriting external changes. Co-authored-by: Gustavo Noronha Silva <gustavo@noronha.dev.br> Co-authored-by: LeoniePhiline <22329650+LeoniePhiline@users.noreply.github.com> Co-authored-by: Pascal Kuthe <pascal.kuthe@semimod.de>	2 years ago
Skyler Hawthorne	a5233cf5ad	disable lsp in integration tests	2 years ago
Ivan Tham	a20a96abdc	Remove apply_transaction helper (#5598)	2 years ago
Michael Davis	66238e8556	Silence dead_code warning on AppBuilder::with_config ... This function is not currently used but is likely to be useful in the future, so this change silences the dead_code warning.	2 years ago
Michael Davis	313579d27c	Remove language-server configuration in integration tests ... This change removes language server configuration from the default languages.toml config for integration tests. No integration-tests currently depend on the availability of a language server but if any future test needs to, it may provide a language server configuration by passing an override into the `test_syntax_conf` helper. Language-servers in integration tests cause false-positive failures when running integration tests in GitHub Actions CI. The Windows runner appears to have `clangd` installed and all OS runners have the `R` binary installed but not the `R` language server package. If a test file created by `tempfile::NamedTempFile` happens to have a file extension of `r`, the test will most likely fail because the R language server will fail to start and will become a broken pipe, meaning that it will fail to shutdown within the timeout, causing a false-positive failure. This happens surprisingly often in practice. Language servers (especially rust-analyzer) also emit unnecessary log output when initializing, which this change silences.	2 years ago
Michael Davis	1243db11a5	Use helix_view::apply_transaction in integration-tests ... `helix_view::apply_transaction` closes over `Document::apply` and `View::apply` to ensure that jumplist entries are updated when a document changes from a transaction. `Document::apply` shouldn't be called directly - this helper function should be used instead.	2 years ago
Skyler Hawthorne	beb3427bfb	improve app close failure display	2 years ago
Skyler Hawthorne	3f07885b35	document should save even if formatter fails	2 years ago
Skyler Hawthorne	57de4e6251	various fixes in write-all path	2 years ago
Skyler Hawthorne	e1f7bdb1d2	fix buffer-close	2 years ago
Skyler Hawthorne	665286c199	factor new Application with file arg to function	2 years ago
Skyler Hawthorne	fac36bc5ea	add test for write-quit happy path	2 years ago
Skyler Hawthorne	7c0bca186c	rename test helpers	2 years ago
Skyler Hawthorne	8d8d389536	rename top level module to satisfy cargo fmt	2 years ago