Vladik Tarasov comments

Results 8 comments of


                                            Vladik Tarasov

fix: change order of node_modules and app in `js_image_layer`

@thesayyn What do you think?

fix: change order of node_modules and app in `js_image_layer`

@thesayyn So I don't see any problem, let's add that change in 2.x.x release? WDYT?

[Bug]: copy_files = True for ts_proto_library doesn't copy d.ts files to source tree

Read again code of `ts_proto_library`. As I understand, generating and copy files on `bazel build proto_grpc` was not even planned (Am I right?) And I need to run `bazel run...

Bazel `info` -> `cquery` invalidate part of cache

@tpasternak You are right. Checked and even just `bazel info` invalidate part of cache, thanks!

Bazel `info` -> `cquery` invalidate part of cache

> Does this repro with `--noexperimental_merged_skyframe_analysis_execution`? I'm wondering if it's the limitation described in this commit description (and afaik not yet lifted since then): [d94816a](https://github.com/bazelbuild/bazel/commit/d94816a68980628e43f509bcd6a49e98984658b5) (although I would concur that...

Bazel `info` -> `cquery` invalidate part of cache

Can we up this? The bug is still reproduced.

Bazel `info` -> `cquery` invalidate part of cache

@reimai > So it might be feasible to look for flags distinct between commands. Reproducing even without flags (

Bazel `info` -> `cquery` invalidate part of cache

Found out, that running `bazel info output_base` doesn't evict cache. `bazel info output_base && bazel cquery //next.js:next && bazel build //next.js:next` works as expected, but running `bazel info` evict part...