{"push_id":3156664182,"size":2,"distinct_size":2,"ref":"refs/heads/master","head":"1d21ad9c46b833d463cb1bb9fb593c6106f0e44a","before":"5d905abbe77295e65c88b50940af4bfb99a48482","commits":[{"sha":"caa3cc8868206f8109e0d633efb09d31e94ef635","author":{"name":"yuuji.yaginuma","email":"16356534c1589fe78070f5640bedf8614007153c@gmail.com"},"message":"Do not add parent directory to file system monitoring\n\n`EventedFileUpdateChecker` will search the parent directory if the\nspecified directory does not exist.\n\nSince `test/mailers/previews` is included in the watch target by default,\nif there is no test directory (e.g. using `rspec`), the Rails root directory\nwill be included in the watch target.\n\n```\n$ rails new app\n$ cd app\n$ ./bin/rails r \"p Rails.application.reloaders.last.send(:directories_to_watch).include?(Rails.root)\"\nfalse\n$ rm -rf test\n$ ./bin/rails r \"p Rails.application.reloaders.last.send(:directories_to_watch).include?(Rails.root)\"\ntrue\n```\n\nThis causes `node_modules` to be included in watch target. Adding parent\ndirectories to watch target may include unexpected directories.\n\nIn order to avoid this, fixed that parents of nonexistent directories are\nnot added to the watch targets, instead checking that the directory\nexists when checking changes.\n\nRelated to #32700.\n\n[Matthew Draper & Yuji Yaginuma]","distinct":true,"url":"https://api.github.com/repos/rails/rails/commits/caa3cc8868206f8109e0d633efb09d31e94ef635"},{"sha":"1d21ad9c46b833d463cb1bb9fb593c6106f0e44a","author":{"name":"Yuji Yaginuma","email":"16356534c1589fe78070f5640bedf8614007153c@gmail.com"},"message":"Merge pull request #33822 from y-yagi/do_not_check_parents_dor_directories\n\nDo no watch parent directory of `dirs`","distinct":true,"url":"https://api.github.com/repos/rails/rails/commits/1d21ad9c46b833d463cb1bb9fb593c6106f0e44a"}]}
{"actor":{"display_login":"y-yagi"}}