diff mbox series

[yocto-metrics,1/2] patch-status: Add missing inactive-upstream status

Message ID CAN21hX7y7j64LM+wp8vV9f6_uKo84auwqk3wCGtJ949VMq5JYQ@mail.gmail.com
State New
Headers show
Series [yocto-metrics,1/2] patch-status: Add missing inactive-upstream status | expand

Commit Message

Alexandre TRUONG July 4, 2024, 1:45 p.m. UTC
From: Alexandre Truong <alexandre.truong@smile.fr>

Adding missing inactive-upstream status to the pie chart and
to upstream status graph.

Signed-off-by: Alexandre Truong <alexandre.truong@smile.fr>
Reviewed-by: Yoann Congal <yoann.congal@smile.fr>
---
 patch-status/index.html | 19 +++++++++++++++----
 1 file changed, 15 insertions(+), 4 deletions(-)

from the original project maintainers to merge the patch.
+            </em></p>
+          </dd>
           <dt><strong>Total</strong>:</dt>
           <dd>
             <p>The total count of patches in the OE-Core meta
directory, regardless of their upstream status.</p>
@@ -435,7 +443,8 @@ SPDX-License-Identifier: MIT
       denied: 'Denied',
       total: 'Total',
       sob: 'Malformed Signed-off-by',
-      upstream_status: 'Malformed Upstream-Status'
+      upstream_status: 'Malformed Upstream-Status',
+      inactive_upstream: 'Inactive-Upstream'
     };
     const branches = ['master', 'scarthgap', 'nanbield',
'mickledore', 'langdale', 'kirkstone', 'honister', 'hardknott',
'gatesgarth', 'dunfell']

@@ -524,7 +533,8 @@ SPDX-License-Identifier: MIT
                     inappropriate: status.inappropriate || 0,
                     accepted: status.accepted || 0,
                     submitted: status.submitted || 0,
-                    denied: status.denied || 0
+                    denied: status.denied || 0,
+                    inactive_upstream: status['inactive-upstream'] || 0
                   }
                 })

@@ -625,7 +635,8 @@ SPDX-License-Identifier: MIT
           { "value": data.inappropriate || 0, "name": "Inappropriate" },
           { "value": data.accepted || 0, "name": "Accepted" },
           { "value": data.submitted || 0, "name": "Submitted" },
-          { "value": data.denied || 0, "name": "Denied" }
+          { "value": data.denied || 0, "name": "Denied" },
+          { "value": data['inactive-upstream'] || 0, "name":
"Inactive-Upstream" }
         ]
         generatePieChart(formatted)
       });
@@ -741,4 +752,4 @@ SPDX-License-Identifier: MIT
   </script>
 </body>

-</html>
\ No newline at end of file
+</html>

Comments

Richard Purdie July 4, 2024, 1:52 p.m. UTC | #1
On Thu, 2024-07-04 at 09:45 -0400, Alexandre Truong via lists.yoctoproject.org wrote:
> From: Alexandre Truong <alexandre.truong@smile.fr>
> 
> Adding missing inactive-upstream status to the pie chart and
> to upstream status graph.
> 
> Signed-off-by: Alexandre Truong <alexandre.truong@smile.fr>
> Reviewed-by: Yoann Congal <yoann.congal@smile.fr>
> ---
>  patch-status/index.html | 19 +++++++++++++++----
>  1 file changed, 15 insertions(+), 4 deletions(-)
> 
> diff --git a/patch-status/index.html b/patch-status/index.html
> index 99ddbbd2..c77db918 100644
> --- a/patch-status/index.html
> +++ b/patch-status/index.html
> @@ -272,6 +272,14 @@ SPDX-License-Identifier: MIT
>              <p><em> Rejected patches could signify issues with patch
> quality, conflicts, or discrepancies between the patch and upstream
> requirements.</em></p>
>            </dd>
> 
> +          <dt><strong>Inactive-Upstream</strong></dt>
> +          <dd>
> +            <p>The upstream is no longer available, due to lack of
> activity for several years.</p>
> +            <p><em>
> +              Inactive-Upstream patches indicate that the original
> source code repository is defunct, with no recent commit or releases,
> and unaddressed bug reports and merge requests.
> +              There is no longer active maintenance or development
> from the original project maintainers to merge the patch.
> +            </em></p>
> +          </dd>
>            <dt><strong>Total</strong>:</dt>
>  
The patches look good but unfortunately they've come through as line
wrapped and don't apply. Could you fix that and resend please?

Thanks,

Richard
diff mbox series

Patch

diff --git a/patch-status/index.html b/patch-status/index.html
index 99ddbbd2..c77db918 100644
--- a/patch-status/index.html
+++ b/patch-status/index.html
@@ -272,6 +272,14 @@  SPDX-License-Identifier: MIT
             <p><em> Rejected patches could signify issues with patch
quality, conflicts, or discrepancies between the patch and upstream
requirements.</em></p>
           </dd>

+          <dt><strong>Inactive-Upstream</strong></dt>
+          <dd>
+            <p>The upstream is no longer available, due to lack of
activity for several years.</p>
+            <p><em>
+              Inactive-Upstream patches indicate that the original
source code repository is defunct, with no recent commit or releases,
and unaddressed bug reports and merge requests.
+              There is no longer active maintenance or development