[wp-trac] [WordPress Trac] #62153: Fix recording Performance Test results

WordPress Trac noreply at wordpress.org
Thu Oct 10 19:16:16 UTC 2024


#62153: Fix recording Performance Test results
-------------------------------------------------+-------------------------
 Reporter:  joemcgill                            |       Owner:  flixos90
     Type:  defect (bug)                         |      Status:  reopened
 Priority:  high                                 |   Milestone:  6.7
Component:  Build/Test Tools                     |     Version:  6.6
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch has-unit-tests fixed-      |     Focuses:
  major                                          |  performance
-------------------------------------------------+-------------------------

Comment (by joemcgill):

 Good point @desrosj, We really only need the new env variable when sending
 the data to the dashboard for commits to trunk, so I wonder if we could
 update the reusable test so that the `CODEVITALS_PROJECT_TOKEN` input is
 not required. The only step that uses that value already has a conditional
 to only run on `trunk` so this should be fine.

 I've opened a [https://github.com/WordPress/wordpress-develop/pull/7550
 new PR] that makes this change.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/62153#comment:26>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list