You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@pagespeed.apache.org by GitBox <gi...@apache.org> on 2020/08/05 20:10:53 UTC

[GitHub] [incubator-pagespeed-mod] oschaaf commented on pull request #2028: Move pagespeed/kernel/base test specifics to test/

oschaaf commented on pull request #2028:
URL: https://github.com/apache/incubator-pagespeed-mod/pull/2028#issuecomment-669475984


   Well, I find it is useful anytime one wants to make an easy (non bazel-based) distinction  between test and production code. 
   One instance of that is running builds of just production code during development, e.g. `bazel build //pagespeed/...`, without remembering build target names. 
   Personally, I also find it helpful to have this separation, as this shrinks down the number of files that go into some of the existing directories, leaning up the listings to code relevant to production. Having said that, this is just personal preference -- if you feel this isn't worth it / shouldn't be done, I'd be happy to close this.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org