Skip to content
Snippets Groups Projects
  1. Dec 09, 2020
  2. Dec 08, 2020
    • Adam Bookatz's avatar
      3a786128
    • Jeff Sharkey's avatar
      Merge changes from topic "dec5" · c2793cf1
      Jeff Sharkey authored
      * changes:
        Improve OWNERS coverage across frameworks/base/.
        Improve OWNERS coverage across frameworks/base/.
        Improve OWNERS coverage across frameworks/base/.
      c2793cf1
    • Jeff Sharkey's avatar
      Improve OWNERS coverage across frameworks/base/. · 52b9862f
      Jeff Sharkey authored
      As general background, OWNERS files expedite code reviews by helping
      code authors quickly find relevant reviewers, and they also ensure
      that stakeholders are involved in code changes in their areas.
      
      Some teams under frameworks/base/ have been using OWNERS files
      successfully for many years, and we're ready to expand them to cover
      more areas.  Here's the historical coverage statistics for the last
      two years of changes before these new OWNERS changes land:
      
      -- 56% of changes are fully covered by OWNERS
      -- 17% of changes are partially covered by OWNERS
      -- 25% of changes have no OWNERS coverage
      
      Working closely with team leads, we've now identified clear OWNERS on
      a per-package basis, and we're using "include" directives whenever
      possible to to simplify future maintenance.  With this extensive
      effort, we've now improved our coverage as follows:
      
      -- 98% of changes are fully covered by OWNERS
      -- 1% of changes are partially covered by OWNERS
      -- 1% of changes have no OWNERS coverage
      
      This specific change is automatically generated by a script that
      identifies relevant "include" directives.
      
      Bug: 174932174
      Test: manual
      Exempt-From-Owner-Approval: refactoring with team leads buy-in
      Merged-In: I3480ddf2fe7ba3dfb922b459d4da01fa17a2c813
      Change-Id: I3480ddf2fe7ba3dfb922b459d4da01fa17a2c813
      52b9862f
    • Jeff Sharkey's avatar
      Improve OWNERS coverage across frameworks/base/. · fab0ab3c
      Jeff Sharkey authored
      As general background, OWNERS files expedite code reviews by helping
      code authors quickly find relevant reviewers, and they also ensure
      that stakeholders are involved in code changes in their areas.
      
      Some teams under frameworks/base/ have been using OWNERS files
      successfully for many years, and we're ready to expand them to cover
      more areas.  Here's the historical coverage statistics for the last
      two years of changes before these new OWNERS changes land:
      
      -- 56% of changes are fully covered by OWNERS
      -- 17% of changes are partially covered by OWNERS
      -- 25% of changes have no OWNERS coverage
      
      Working closely with team leads, we've now identified clear OWNERS on
      a per-package basis, and we're using "include" directives whenever
      possible to to simplify future maintenance.  With this extensive
      effort, we've now improved our coverage as follows:
      
      -- 98% of changes are fully covered by OWNERS
      -- 1% of changes are partially covered by OWNERS
      -- 1% of changes have no OWNERS coverage
      
      This specific change is automatically generated by a script from
      detailed ownership information confirmed by team leads.
      
      Bug: 174932174
      Test: manual
      Exempt-From-Owner-Approval: refactoring with team leads buy-in
      Merged-In: I9789c97c1de8e5d962b48c29c57d82fe83729eba
      Change-Id: I9789c97c1de8e5d962b48c29c57d82fe83729eba
      fab0ab3c
    • Jeff Sharkey's avatar
      Improve OWNERS coverage across frameworks/base/. · 061c2ec3
      Jeff Sharkey authored
      As general background, OWNERS files expedite code reviews by helping
      code authors quickly find relevant reviewers, and they also ensure
      that stakeholders are involved in code changes in their areas.
      
      Some teams under frameworks/base/ have been using OWNERS files
      successfully for many years, and we're ready to expand them to cover
      more areas.  Here's the historical coverage statistics for the last
      two years of changes before these new OWNERS changes land:
      
      -- 56% of changes are fully covered by OWNERS
      -- 17% of changes are partially covered by OWNERS
      -- 25% of changes have no OWNERS coverage
      
      Working closely with team leads, we've now identified clear OWNERS on
      a per-package basis, and we're using "include" directives whenever
      possible to to simplify future maintenance.  With this extensive
      effort, we've now improved our coverage as follows:
      
      -- 98% of changes are fully covered by OWNERS
      -- 1% of changes are partially covered by OWNERS
      -- 1% of changes have no OWNERS coverage
      
      This specific change begins defining top-level OWNERS lists,
      including a general catch-all for string translations.
      
      Bug: 174932174
      Test: manual
      Exempt-From-Owner-Approval: refactoring with team leads buy-in
      Change-Id: Ie7ac3302d40a717fa048115cca2ea4359de64959
      061c2ec3
    • Treehugger Robot's avatar
    • Fiona Campbell's avatar
    • Lorenzo Colitti's avatar
  3. Dec 07, 2020
  4. Dec 06, 2020
Loading