老赵
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修改起来简单可以一次通过，否则来来回回也是很花时间。