-
-
Notifications
You must be signed in to change notification settings - Fork 13.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ast-grep: 0.24.1 -> 0.26.3 #327101
ast-grep: 0.24.1 -> 0.26.3 #327101
Conversation
Result of 1 package failed to build:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This fails to build for me with
error: builder for '/nix/store/ghqpmgp2grl3nhghh0dhc5r167yvkgnc-ast-grep-0.25.0.drv' failed with exit code 101;
last 25 log lines:
> test verify::test_case::tests::test_verify_snapshot_without_existing ... ok
> test verify::test_case::tests::test_verify_rule ... ok
> test verify::test_case::tests::test_invalid ... ok
> test print::interactive_print::test::test_open_editor ... ok
> test run::test::test_run_with_strictness ... ok
> test run::test::test_run_with_specific_lang ... ok
> test scan::test::test_scan_with_inline_rules_diff ... ok
> test scan::test::test_scan_with_inline_rules ... ok
> test scan::test::test_run_with_config ... ok
> test run::test::test_run_with_pattern ... ok
>
> failures:
>
> ---- lang::lang_globs::test::test_merge_with_globs stdout ----
> thread 'lang::lang_globs::test::test_merge_with_globs' panicked at crates/cli/src/lang/lang_globs.rs:166:5:
> assertion failed: html_types.matched("a.vue", false).is_whitelist()
> note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
>
>
> failures:
> lang::lang_globs::test::test_merge_with_globs
>
> test result: FAILED. 77 passed; 1 failed; 1 ignored; 0 measured; 1 filtered out; finished in 0.05s
>
> error: test failed, to rerun pass `-p ast-grep --lib`
For full logs, run 'nix log /nix/store/ghqpmgp2grl3nhghh0dhc5r167yvkgnc-ast-grep-0.25.0.drv'.
There is 0.25.2 now, how do we get the bot to update to latest? |
180c151
to
565ef29
Compare
Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/ast-grep/versions. meta.description for ast-grep is: Fast and polyglot tool for code searching, linting, rewriting at large scale meta.homepage for ast-grep is: https://ast-grep.github.io/ meta.changelog for ast-grep is: https://github.com/ast-grep/ast-grep/blob/0.25.3/CHANGELOG.md Updates performed
To inspect upstream changesImpactChecks done
Rebuild report (if merged into master) (click to expand)
Instructions to test this update (click to expand)Either download from Cachix:
(The Cachix cache is only trusted for this store-path realization.) Or, build yourself:
Or:
After you've downloaded or built it, look at the files and if there are any, run the binaries:
Pre-merge build resultsWe have automatically built all packages that will get rebuilt due to This gives evidence on whether the upgrade will break dependent packages. Result of 1 package built:
Maintainer pingscc @montchr @Lord-Valen @cafkafk for testing. |
565ef29
to
b5355ba
Compare
Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/ast-grep/versions. meta.description for ast-grep is: Fast and polyglot tool for code searching, linting, rewriting at large scale meta.homepage for ast-grep is: https://ast-grep.github.io/ meta.changelog for ast-grep is: https://github.com/ast-grep/ast-grep/blob/0.25.6/CHANGELOG.md Updates performed
To inspect upstream changesImpactChecks done
Rebuild report (if merged into master) (click to expand)
Instructions to test this update (click to expand)Either download from Cachix:
(The Cachix cache is only trusted for this store-path realization.) Or, build yourself:
Or:
After you've downloaded or built it, look at the files and if there are any, run the binaries:
Pre-merge build resultsWe have automatically built all packages that will get rebuilt due to This gives evidence on whether the upgrade will break dependent packages. Result of 1 package built:
Maintainer pingscc @montchr @Lord-Valen @cafkafk for testing. |
b5355ba
to
2db942f
Compare
Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/ast-grep/ast-grep/releases. meta.description for ast-grep is: Fast and polyglot tool for code searching, linting, rewriting at large scale meta.homepage for ast-grep is: https://ast-grep.github.io/ meta.changelog for ast-grep is: https://github.com/ast-grep/ast-grep/blob/0.26.1/CHANGELOG.md Updates performed
To inspect upstream changesImpactChecks done
Rebuild report (if merged into master) (click to expand)
Instructions to test this update (click to expand)Either download from Cachix:
(The Cachix cache is only trusted for this store-path realization.) Or, build yourself:
Or:
After you've downloaded or built it, look at the files and if there are any, run the binaries:
Pre-merge build resultsWe have automatically built all packages that will get rebuilt due to This gives evidence on whether the upgrade will break dependent packages. Result of 1 package built:
Maintainer pingscc @montchr @Lord-Valen @cafkafk for testing. Add a 👍 reaction to pull requests you find important. |
@cafkafk @Lord-Valen @montchr This builds and runs fine for me now (0.26.1) on aarch64 darwin. Could it be merged? |
2db942f
to
457ec7e
Compare
Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/ast-grep/ast-grep/releases. meta.description for ast-grep is: Fast and polyglot tool for code searching, linting, rewriting at large scale meta.homepage for ast-grep is: https://ast-grep.github.io/ meta.changelog for ast-grep is: https://github.com/ast-grep/ast-grep/blob/0.26.2/CHANGELOG.md Updates performed
To inspect upstream changesImpactChecks done
Rebuild report (if merged into master) (click to expand)
Instructions to test this update (click to expand)Either download from Cachix:
(The Cachix cache is only trusted for this store-path realization.) Or, build yourself:
Or:
After you've downloaded or built it, look at the files and if there are any, run the binaries:
Pre-merge build resultsWe have automatically built all packages that will get rebuilt due to This gives evidence on whether the upgrade will break dependent packages. Result of 1 package built:
Maintainer pingscc @montchr @Lord-Valen @cafkafk for testing. Tip As a maintainer, if your package is located under Add a 👍 reaction to pull requests you find important. |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/prs-ready-for-review/3032/4454 |
Result of 1 package built:
|
457ec7e
to
9a8c62f
Compare
Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/ast-grep/ast-grep/releases. meta.description for ast-grep is: Fast and polyglot tool for code searching, linting, rewriting at large scale meta.homepage for ast-grep is: https://ast-grep.github.io/ meta.changelog for ast-grep is: https://github.com/ast-grep/ast-grep/blob/0.26.3/CHANGELOG.md Updates performed
To inspect upstream changesImpactChecks done
Rebuild report (if merged into master) (click to expand)
Instructions to test this update (click to expand)Either download from Cachix:
(The Cachix cache is only trusted for this store-path realization.) Or, build yourself:
Or:
After you've downloaded or built it, look at the files and if there are any, run the binaries:
Pre-merge build resultsWe have automatically built all packages that will get rebuilt due to This gives evidence on whether the upgrade will break dependent packages. Result of 1 package built:
Maintainer pingscc @montchr @Lord-Valen @cafkafk for testing. Tip As a maintainer, if your package is located under Add a 👍 reaction to pull requests you find important. |
Result of 1 package built:
|
Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/ast-grep/ast-grep/releases.
meta.description for ast-grep is: Fast and polyglot tool for code searching, linting, rewriting at large scale
meta.homepage for ast-grep is: https://ast-grep.github.io/
meta.changelog for ast-grep is: https://github.com/ast-grep/ast-grep/blob/0.25.0/CHANGELOG.md
Updates performed
To inspect upstream changes
Release on GitHub
Compare changes on GitHub
Impact
Checks done
passthru.tests
, if any, passedRebuild report (if merged into master) (click to expand)
Instructions to test this update (click to expand)
Either download from Cachix:
(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the
trusted-users
list or you can usesudo
since root is effectively trusted.Or, build yourself:
Or:
After you've downloaded or built it, look at the files and if there are any, run the binaries:
Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of
nixpkgs-review
run on x86_64-linux 11 package built:
Maintainer pings
cc @montchr @Lord-Valen @cafkafk for testing.