from json --objects
streaming (#12949)"}},{"before":"1a7099e6f8ecfec8b8e3e228911c9b9660be923d","after":"2ef879230783601b6405948633eb618cd4ff393f","ref":"refs/heads/dependabot/cargo/polars-ops-0.40.0","pushedAt":"2024-05-24T23:11:44.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"dependabot[bot]","name":null,"path":"/apps/dependabot","primaryAvatarUrl":"https://avatars.githubusercontent.com/in/29110?s=80&v=4"},"commit":{"message":"---\nupdated-dependencies:\n- dependency-name: polars-ops\n dependency-type: direct:production\n update-type: version-update:semver-minor\n...\n\nSigned-off-by: dependabot[bot] std::env::set_current_dir()
call from `EngineState::…"}},{"before":null,"after":"6f50efb0a4b2d5d9390d994e76c8110627e3dc30","ref":"refs/heads/revert-12922-no-set-current-dir","pushedAt":"2024-05-24T15:42:14.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"fdncred","name":"Darren Schroeder","path":"/fdncred","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/343840?s=80&v=4"},"commit":{"message":"Revert \"Remove `std::env::set_current_dir()` call from `EngineState::merge_en…\"\n\nThis reverts commit 7ede90cba50e4dad62a2ff503a2566c82a4935ea.","shortMessageHtmlLink":"Revert \"Remove std::env::set_current_dir()
call from `EngineState::…"}},{"before":"0b5a4c0d951622e60c318ecaa66f15ab2c80eeaa","after":"bf07806b1bc96eebcaa57456c97035a6bce68ef7","ref":"refs/heads/main","pushedAt":"2024-05-23T20:38:47.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"IanManske","name":"Ian Manske","path":"/IanManske","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/34794997?s=80&v=4"},"commit":{"message":"Use `cwd` in `grid` (#12947)\n\n# Description\r\nFixes #12946. The `grid` command does not use the cwd when trying to get\r\nthe icon or color for a file/path.","shortMessageHtmlLink":"Use cwd
in grid
(#12947)"}},{"before":"f53aa6fcbfe53563d8e04d5f3717c5890860f5da","after":"0b5a4c0d951622e60c318ecaa66f15ab2c80eeaa","ref":"refs/heads/main","pushedAt":"2024-05-23T13:51:39.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"fdncred","name":"Darren Schroeder","path":"/fdncred","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/343840?s=80&v=4"},"commit":{"message":"`explore` refactoring+clarification (#12940)\n\nAnother very boring PR cleaning up and documenting some of `explore`'s\r\ninnards. Mostly renaming things that I found confusing or vague when\r\nreading through the code, also adding some comments.","shortMessageHtmlLink":"explore
refactoring+clarification (#12940)"}},{"before":"2612a167e31fce874a7faaaecf2222a38c03c6c8","after":"f53aa6fcbfe53563d8e04d5f3717c5890860f5da","ref":"refs/heads/main","pushedAt":"2024-05-23T13:51:02.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"fdncred","name":"Darren Schroeder","path":"/fdncred","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/343840?s=80&v=4"},"commit":{"message":"fix std help (#12943)\n\n# Description\r\nFixes: #12941\r\n\r\n~~The issue is cause by some columns(is_builtin, is_plugin, is_custom,\r\nis_keyword) are removed in #10023~~\r\nEdit: I'm wrong\r\n\r\n# Tests + Formatting\r\nAdded one test for `std help`","shortMessageHtmlLink":"fix std help (#12943)"}},{"before":"c7097ca93761ea2062fef2271ebbdc4a10d67bce","after":"2612a167e31fce874a7faaaecf2222a38c03c6c8","ref":"refs/heads/main","pushedAt":"2024-05-23T05:53:55.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"WindSoilder","name":"Wind","path":"/WindSoilder","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/22256154?s=80&v=4"},"commit":{"message":"Remove list support in `with-env` (#12939)\n\n# Description\r\nFollowing from #12523, this PR removes support for lists of environments\r\nvariables in the `with-env` command. Rather, only records will be\r\nsupported now.\r\n\r\n# After Submitting\r\nUpdate examples using the list form in the docs and book.","shortMessageHtmlLink":"Remove list support in with-env
(#12939)"}},{"before":"58cf0c56f8cefc3c1bf9d4a39d3fdb19890b8681","after":"c7097ca93761ea2062fef2271ebbdc4a10d67bce","ref":"refs/heads/main","pushedAt":"2024-05-23T03:06:14.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"rgwood","name":"Reilly Wood","path":"/rgwood","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/26268125?s=80&v=4"},"commit":{"message":"`explore` cleanup: remove+move binary viewer config (#12920)\n\nSmall change, removing 4 more configuration options from `explore`'s\r\nbinary viewer:\r\n\r\n1. `show_index`\r\n2. `show_data`\r\n3. `show_ascii`\r\n4. `show_split`\r\n\r\nThese controlled whether the 3 columns in the binary viewer (index, hex\r\ndata, ASCII) and the pipe separator (`|`) in between them are shown. I\r\ndon't think we need this level of configurability until the `explore`\r\ncommand is more mature, and maybe even not then; we can just show them\r\nall.\r\n\r\nI think it's very unlikely that anyone is using these configuration\r\npoints.\r\n\r\nAlso, the row offset (e.g. how many rows we have scrolled down) was\r\nbeing stored in config/settings when it's arguably not config; more like\r\ninternal state of the binary viewer. I moved it to a more appropriate\r\nlocation and renamed it.","shortMessageHtmlLink":"explore
cleanup: remove+move binary viewer config (#12920)"}},{"before":"6c649809d38a687cd63b59ab028b7f30207e5663","after":"58cf0c56f8cefc3c1bf9d4a39d3fdb19890b8681","ref":"refs/heads/main","pushedAt":"2024-05-23T02:47:06.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"WindSoilder","name":"Wind","path":"/WindSoilder","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/22256154?s=80&v=4"},"commit":{"message":"add some completion tests (#12908)\n\n# Description\r\n```nushell\r\n❯ ls\r\n╭───┬───────┬──────┬──────┬──────────╮\r\n│ # │ name │ type │ size │ modified │\r\n├───┼───────┼──────┼──────┼──────────┤\r\n│ 0 │ a.txt │ file │ 0 B │ now │\r\n╰───┴───────┴──────┴──────┴──────────╯\r\n\r\n❯ ls a.\r\nNO RECORDS FOUND\r\n```\r\n\r\nThere is a completion issue on previous version, I think @amtoine have\r\nreproduced it before. But currently I can't reproduce it on latest main.\r\nTo avoid such regression, I added some tests for completion.\r\n\r\n---------\r\n\r\nCo-authored-by: Antoine Stevan <44101798+amtoine@users.noreply.github.com>","shortMessageHtmlLink":"add some completion tests (#12908)"}},{"before":"64afb52ffaa68ac761b5f5d05d51d3fb1547084b","after":"6c649809d38a687cd63b59ab028b7f30207e5663","ref":"refs/heads/main","pushedAt":"2024-05-23T02:05:27.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"IanManske","name":"Ian Manske","path":"/IanManske","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/34794997?s=80&v=4"},"commit":{"message":"Rewrite run_external.rs (#12921)\n\nThis PR is a complete rewrite of `run_external.rs`. The main goal of the\r\nrewrite is improving readability, but it also fixes some bugs related to\r\nargument handling and the PATH variable (fixes\r\nhttps://github.com/nushell/nushell/issues/6011).\r\n\r\nI'll discuss some technical details to make reviewing easier.\r\n\r\n## Argument handling\r\n\r\nQuoting arguments for external commands is hard. Like, *really* hard.\r\nWe've had more than a dozen issues and PRs dedicated to quoting\r\narguments (see Appendix) but the current implementation is still buggy.\r\n\r\nHere's a demonstration of the buggy behavior:\r\n\r\n```nu\r\nlet foo = \"'bar'\"\r\n^touch $foo # This creates a file named `bar`, but it should be `'bar'`\r\n^touch ...[ \"'bar'\" ] # Same\r\n```\r\n\r\nI'll describe how this PR deals with argument handling.\r\n\r\nFirst, we'll introduce the concept of **bare strings**. Bare strings are\r\n**string literals** that are either **unquoted** or **quoted by\r\nbackticks** [^1]. Strings within a list literal are NOT considered bare\r\nstrings, even if they are unquoted or quoted by backticks.\r\n\r\nWhen a bare string is used as an argument to external process, we need\r\nto perform tilde-expansion, glob-expansion, and inner-quotes-removal, in\r\nthat order. \"Inner-quotes-removal\" means transforming from\r\n`--option=\"value\"` into `--option=value`.\r\n\r\n## `.bat` files and CMD built-ins\r\n\r\nOn Windows, `.bat` files and `.cmd` files are considered executable, but\r\nthey need `CMD.exe` as the interpreter. The Rust standard library\r\nsupports running `.bat` files directly and will spawn `CMD.exe` under\r\nthe hood (see\r\n[documentation](https://doc.rust-lang.org/std/process/index.html#windows-argument-splitting)).\r\nHowever, other extensions are not supported [^2].\r\n\r\nNushell also supports a selected number of CMD built-ins. The problem\r\nwith CMD is that it uses a different set of quoting rules. Correctly\r\nquoting for CMD requires using\r\n[Command::raw_arg()](https://doc.rust-lang.org/std/os/windows/process/trait.CommandExt.html#tymethod.raw_arg)\r\nand manually quoting CMD special characters, on top of quoting from the\r\nNushell side. ~~I decided that this is too complex and chose to reject\r\nspecial characters in CMD built-ins instead [^3]. Hopefully this will\r\nnot affact real-world use cases.~~ I've implemented escaping that works\r\nreasonably well.\r\n\r\n## `which-support` feature\r\n\r\nThe `which` crate is now a hard dependency of `nu-command`, making the\r\n`which-support` feature essentially useless. The `which` crate is\r\nalready a hard dependency of `nu-cli`, and we should consider removing\r\nthe `which-support` feature entirely.\r\n\r\n## Appendix\r\n\r\nHere's a list of quoting-related issues and PRs in rough chronological\r\norder.\r\n\r\n* https://github.com/nushell/nushell/issues/4609\r\n* https://github.com/nushell/nushell/issues/4631\r\n* https://github.com/nushell/nushell/issues/4601\r\n * https://github.com/nushell/nushell/pull/5846\r\n* https://github.com/nushell/nushell/issues/5978\r\n * https://github.com/nushell/nushell/pull/6014\r\n* https://github.com/nushell/nushell/issues/6154\r\n * https://github.com/nushell/nushell/pull/6161\r\n* https://github.com/nushell/nushell/issues/6399\r\n * https://github.com/nushell/nushell/pull/6420\r\n * https://github.com/nushell/nushell/pull/6426\r\n* https://github.com/nushell/nushell/issues/6465\r\n* https://github.com/nushell/nushell/issues/6559\r\n * https://github.com/nushell/nushell/pull/6560\r\n\r\n[^1]: The idea that backtick-quoted strings act like bare strings was\r\nintroduced by Kubouch and briefly mentioned in [the language\r\nreference](https://www.nushell.sh/lang-guide/chapters/strings_and_text.html#backtick-quotes).\r\n\r\n[^2]: The documentation also said \"running .bat scripts in this way may\r\nbe removed in the future and so should not be relied upon\", which is\r\nanother reason to move away from this. But again, quoting for CMD is\r\nhard.\r\n\r\n[^3]: If anyone wants to try, the best resource I found on the topic is\r\n[this](https://daviddeley.com/autohotkey/parameters/parameters.htm).","shortMessageHtmlLink":"Rewrite run_external.rs (#12921)"}},{"before":"8ca51a8cb681169a340661d2cd041b7b5b2ec727","after":null,"ref":"refs/heads/kubouch-patch-1","pushedAt":"2024-05-22T21:37:22.000Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"fdncred","name":"Darren Schroeder","path":"/fdncred","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/343840?s=80&v=4"}},{"before":"ac4125f8edd5b8f3f68aa653d03d57b2d25f1da1","after":"64afb52ffaa68ac761b5f5d05d51d3fb1547084b","ref":"refs/heads/main","pushedAt":"2024-05-22T21:24:22.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"IanManske","name":"Ian Manske","path":"/IanManske","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/34794997?s=80&v=4"},"commit":{"message":"Fix leftover wrong column name (#12937)\n\n# Description\r\n\r\nSmall fixup for https://github.com/nushell/nushell/pull/12930","shortMessageHtmlLink":"Fix leftover wrong column name (#12937)"}},{"before":null,"after":"8ca51a8cb681169a340661d2cd041b7b5b2ec727","ref":"refs/heads/kubouch-patch-1","pushedAt":"2024-05-22T21:12:27.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"kubouch","name":"Jakub Žádník","path":"/kubouch","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/25571562?s=80&v=4"},"commit":{"message":"Fix leftover wrong column name","shortMessageHtmlLink":"Fix leftover wrong column name"}},{"before":"7ede90cba50e4dad62a2ff503a2566c82a4935ea","after":"ac4125f8edd5b8f3f68aa653d03d57b2d25f1da1","ref":"refs/heads/main","pushedAt":"2024-05-22T17:00:58.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kubouch","name":"Jakub Žádník","path":"/kubouch","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/25571562?s=80&v=4"},"commit":{"message":"fix range semantic in detect_columns, str substring, str index-of (#12894)\n\n# Description\r\nFixes: https://github.com/nushell/nushell/issues/7761\r\n\r\nIt's still unsure if we want to change the `range semantic` itself, but\r\nit's good to keep range semantic consistent between nushell commands.\r\n\r\n# User-Facing Changes\r\n### Before\r\n```nushell\r\n❯ \"abc\" | str substring 1..=2\r\nb\r\n```\r\n### After\r\n```nushell\r\n❯ \"abc\" | str substring 1..=2\r\nbc\r\n```\r\n\r\n# Tests + Formatting\r\nAdjust tests to fit new behavior","shortMessageHtmlLink":"fix range semantic in detect_columns, str substring, str index-of (#1…"}},{"before":"75689ec98aa7037f1101844ea16f1934c51ff706","after":"7ede90cba50e4dad62a2ff503a2566c82a4935ea","ref":"refs/heads/main","pushedAt":"2024-05-22T16:58:27.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kubouch","name":"Jakub Žádník","path":"/kubouch","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/25571562?s=80&v=4"},"commit":{"message":"Remove `std::env::set_current_dir()` call from `EngineState::merge_env()` (#12922)\n\nAs discussed in https://github.com/nushell/nushell/pull/12749, we no\r\nlonger need to call `std::env::set_current_dir()` to sync `$env.PWD`\r\nwith the actual working directory. This PR removes the call from\r\n`EngineState::merge_env()`.","shortMessageHtmlLink":"Remove std::env::set_current_dir()
call from `EngineState::merge_en…"}},{"before":"7de513a4e04d4654314a02c60a1a415fc5f348a0","after":"75689ec98aa7037f1101844ea16f1934c51ff706","ref":"refs/heads/main","pushedAt":"2024-05-22T16:56:51.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"kubouch","name":"Jakub Žádník","path":"/kubouch","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/25571562?s=80&v=4"},"commit":{"message":"Small improvements to `debug profile` (#12930)\n\n\r\n\r\n# Description\r\n\r\n\r\n1. With the `-l` flag, `debug profile` now collects files and line\r\nnumbers of profiled pipeline elements\r\n\r\n![profiler_lines](https://github.com/nushell/nushell/assets/25571562/b400a956-d958-4aff-aa4c-7e65da3f78fa)\r\n\r\n2. Error from the profiled closure will be reported instead of silently\r\nignored.\r\n\r\n![profiler_lines_error](https://github.com/nushell/nushell/assets/25571562/54f7ad7a-06a3-4d56-92c2-c3466917bee8)\r\n\r\n\r\n# User-Facing Changes\r\n\r\n\r\nNew `--lines(-l)` flag to `debug profile`. The command will also fail if\r\nthe profiled closure fails, so technically it is a breaking change.\r\n\r\n# Tests + Formatting\r\n\r\n\r\n# After Submitting\r\n\r\n\r\n---------\r\n\r\nCo-authored-by: Ian Manske debug profile
(#12930)"}},{"before":"758c5d447aeccdf75fa95dd85b6c661113935df0","after":"7de513a4e04d4654314a02c60a1a415fc5f348a0","ref":"refs/heads/main","pushedAt":"2024-05-22T16:55:24.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"IanManske","name":"Ian Manske","path":"/IanManske","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/34794997?s=80&v=4"},"commit":{"message":"Implement streaming I/O for CSV and TSV commands (#12918)\n\n# Description\r\n\r\nImplements streaming for:\r\n\r\n- `from csv`\r\n- `from tsv`\r\n- `to csv`\r\n- `to tsv`\r\n\r\nvia the new string-typed ByteStream support.\r\n\r\n# User-Facing Changes\r\nCommands above. Also:\r\n\r\n- `to csv` and `to tsv` now have `--columns ps
command on FreeBSD, NetBSD, and OpenBSD (#12892"}},{"before":"8a5a9ef103b3aa7ed1aa9b8c83c3ae38bf11d28a","after":null,"ref":"refs/heads/dependabot/cargo/shadow-rs-0.28.0","pushedAt":"2024-05-22T07:59:39.000Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"dependabot[bot]","name":null,"path":"/apps/dependabot","primaryAvatarUrl":"https://avatars.githubusercontent.com/in/29110?s=80&v=4"}},{"before":"3cf150727c0491ede29f0902bfbd0d410a589184","after":"d7e75c0b70570096cbaa20e61adc174ef5a135b9","ref":"refs/heads/main","pushedAt":"2024-05-22T07:59:33.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"WindSoilder","name":"Wind","path":"/WindSoilder","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/22256154?s=80&v=4"},"commit":{"message":"Bump shadow-rs from 0.27.1 to 0.28.0 (#12932)\n\nBumps [shadow-rs](https://github.com/baoyachi/shadow-rs) from 0.27.1 to\r\n0.28.0.\r\nSourced from shadow-rs's\r\nreleases.
\r\n\r\n\r\nfix cargo clippy
\r\n\r\nThx
\r\n@qartik
ba9f8b0
\r\nUpdate Cargo.tomld1b724c
\r\nMerge pull request #160\r\nfrom qartik/patch-1505108d
\r\nAllow missing_docs for deprecated CLAP_VERSION constantSourced from actions/checkout's\r\nreleases.
\r\n\r\n\r\nv4.1.6
\r\nWhat's Changed
\r\n\r\n
\r\n- Check platform to set archive extension appropriately by
\r\n@cory-miller
in\r\nactions/checkout#1732- Update for 4.1.6 release by
\r\n@cory-miller
in\r\nactions/checkout#1733Full Changelog: https://github.com/actions/checkout/compare/v4.1.5...v4.1.6
\r\n
Sourced from actions/checkout's\r\nchangelog.
\r\n\r\n\r\nv4.1.6
\r\n\r\n
\r\n- Check platform to set archive extension appropriately by
\r\n@cory-miller
in\r\nactions/checkout#1732
a5ac7e5
\r\nUpdate for 4.1.6 release (#1733)24ed1a3
\r\nCheck platform for extension (#1732)