LeetCode专题 分而治之
老赵
2019-11-06 12:05:55 发布
我在八月七号提交给dotnet/corefx的性能优化终于被merge了,这意味着不需要等dotnet把几个repo合并之后再重新提交新的PR了,省了不少时间。接下来dotnet会把几个repo给合并,例如dotnet/coreclr和dotnet/corefx会被合并成dotnet/runtime,这样就避免很多提交时的麻烦。

例如我给dotnet提交的第一个PR(不是上面那个)是修复一个很简单的bug,但是这个bug的代码在coreclr里,但单元测试在corefx里,于是我必须在coreclr里提交代码修改PR,并在corefx里同时提交单元测试修改PR。两个PR都被review通过后,coreclr的代码修改PR会被先merge,等到自动同步到corefx后,再merge单元测试PR,否则corefx里新的单元测试就会出错。

分开的repo在代码修改时也很麻烦,例如同样是上面的bug,我需要先在corefx里写新的单元测试,然后编译运行测试显示失败。然后修复bug并编译coreclr,再基于本地的coreclr重新编译corefx,最后运行测试显示成功。幸好这个bug修改起来简单可以一次通过,否则来来回回也是很花时间。