Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: update pr workflow #615

Merged
merged 3 commits into from
Jul 14, 2023
Merged

chore: update pr workflow #615

merged 3 commits into from
Jul 14, 2023

Conversation

patrickhousley
Copy link
Contributor

@patrickhousley patrickhousley commented Jul 13, 2023

Workflow updates to provide faster feedback on pull requests, cleanup nightly jobs, and make re-usable workflows for jil testing.

Overview

Workflow updates:

  • Pull Request Checks
    • Jobs not needing secrets have been moved to be ran on the pull_request event. The remaining jobs will run on manual dispatch.
    • Updated the WDIO Coverage and WDIO jobs so they run in parallel. Keeping the wdio coverage execution in a separate job so it runs fast and isolated.
    • Added jil unit and functional tests to run after all wdio tests have finished.
  • JIL: All Browsers
    • New reusable workflow that calls JIL: Single Browser for each browser and collection (unit, functional).
  • JIL: Single Browser
    • New reusable workflow that runs a single collection of jil tests for a specified browser.
  • Main Push
    • Updated workflow to prevent it from running multiple times simultaneously. If multiple PRs are merged back to back to main, only the workflow start for the last merge will finish executing. All others will be cancelled.
  • Nightly
    • Added jil smoke and polyfill jobs to this workflow and deleted the old workflow files.
  • WDIO: All browsers
    • Fixed the ref inputs not passing through when using workflow call

Related Issue(s)

https://issues.newrelic.com/browse/NR-137546

Testing

N/A

@codecov
Copy link

codecov bot commented Jul 13, 2023

Codecov Report

Merging #615 (93ec48c) into main (8a6f319) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #615   +/-   ##
=======================================
  Coverage   67.01%   67.01%           
=======================================
  Files         129      129           
  Lines        5990     5990           
  Branches     1140     1140           
=======================================
  Hits         4014     4014           
  Misses       1606     1606           
  Partials      370      370           
Flag Coverage Δ
jest-component 20.98% <ø> (ø)
jest-unit 22.35% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@github-actions
Copy link

github-actions bot commented Jul 13, 2023

Asset Size Report

Merging this pull request will result in the following CDN asset size changes:

Asset Name Previous Size New Size Diff
nr-loader-spa.min 50.62 kB / 17.25 kB (gzip) 50.61 kB / 17.25 kB (gzip) -0.03% / -0.05% (gzip)
nr-loader-full.min 44.7 kB / 15.48 kB (gzip) 44.69 kB / 15.47 kB (gzip) -0.03% / -0.06% (gzip)
nr-loader-rum.min 26.54 kB / 9.73 kB (gzip) 26.53 kB / 9.72 kB (gzip) -0.05% / -0.12% (gzip)
nr-loader-spa-polyfills.min 123.16 kB / 39.38 kB (gzip) 123.15 kB / 39.36 kB (gzip) -0.01% / -0.04% (gzip)
nr-loader-full-polyfills.min 115.68 kB / 37.44 kB (gzip) 115.66 kB / 37.43 kB (gzip) -0.01% / -0.03% (gzip)
nr-loader-rum-polyfills.min 94.63 kB / 31.26 kB (gzip) 94.62 kB / 31.24 kB (gzip) -0.01% / -0.04% (gzip)
nr-loader-worker.min 41.26 kB / 14.21 kB (gzip) 41.24 kB / 14.2 kB (gzip) -0.03% / -0.05% (gzip)

Merging this pull request will result in the following NPM package consumer size changes:

Asset Name Previous Size New Size Diff
Browser Agent 50.76 kB / 17.13 kB (gzip) 50.75 kB / 17.13 kB (gzip) -0.02% / -0.03% (gzip)
Custom Lite Agent 26.69 kB / 9.65 kB (gzip) 26.68 kB / 9.64 kB (gzip) -0.04% / -0.1% (gzip)
Custom Pro Agent 44.76 kB / 15.29 kB (gzip) 44.75 kB / 15.29 kB (gzip) -0.02% / -0.03% (gzip)
Custom SPA Agent 50.64 kB / 17.1 kB (gzip) 50.63 kB / 17.09 kB (gzip) -0.02% / -0.03% (gzip)
Worker Agent 304.78 kB / 94.4 kB (gzip) 303.96 kB / 94.23 kB (gzip) -0.27% / -0.19% (gzip)
Other Standard CDN Assets

Released Assets

Asset Name Asset Size
recorder.1b18459f.min.js 166.07 kB / 52.32 kB (gzip)
spa-aggregate.68f643e9.min.js 20.93 kB / 7.42 kB (gzip)
page_view_timing-aggregate.d7fbc4fb.min.js 14.97 kB / 5.43 kB (gzip)
860.2d6ff735.min.js 14.36 kB / 5.33 kB (gzip)
session_trace-aggregate.973a2a56.min.js 12.24 kB / 4.53 kB (gzip)
page_view_event-aggregate.c8b08262.min.js 11.1 kB / 4.19 kB (gzip)
jserrors-aggregate.3a914a23.min.js 9.58 kB / 3.67 kB (gzip)
metrics-aggregate.8f3b1670.min.js 8.56 kB / 2.93 kB (gzip)
646.fd5f577f.min.js 8.04 kB / 3.32 kB (gzip)
session_replay-aggregate.29998cae.min.js 7.94 kB / 2.91 kB (gzip)
ajax-aggregate.449a907c.min.js 7.13 kB / 3.06 kB (gzip)
compressor.ae9f91a8.min.js 7.09 kB / 3.57 kB (gzip)
page_action-aggregate.da8a4bbf.min.js 4.73 kB / 1.92 kB (gzip)
async-api.30bd804e.min.js 2.92 kB / 1.46 kB (gzip)
session-manager.2a64278a.min.js 1.47 kB / 740 B (gzip)
lazy-feature-loader.94d9d1a8.min.js 1.17 kB / 498 B (gzip)

Built Assets

Asset Name Asset Size
recorder.1b18459f.min.js 166.07 kB / 52.32 kB (gzip)
spa-aggregate.e6b09d52.min.js 20.93 kB / 7.42 kB (gzip)
page_view_timing-aggregate.bd6de33a.min.js 14.97 kB / 5.43 kB (gzip)
860.2d6ff735.min.js 14.36 kB / 5.33 kB (gzip)
session_trace-aggregate.bb839bab.min.js 12.24 kB / 4.52 kB (gzip)
page_view_event-aggregate.cced4cf0.min.js 11.1 kB / 4.18 kB (gzip)
jserrors-aggregate.49e41428.min.js 9.58 kB / 3.67 kB (gzip)
metrics-aggregate.3dc53903.min.js 8.56 kB / 2.93 kB (gzip)
148.1a20d5fe.min.js 7.89 kB / 3.29 kB (gzip)
session_replay-aggregate.cf0efb30.min.js 7.28 kB / 2.75 kB (gzip)
ajax-aggregate.998ef92b.min.js 7.13 kB / 3.06 kB (gzip)
compressor.ae9f91a8.min.js 7.09 kB / 3.57 kB (gzip)
page_action-aggregate.ac76d497.min.js 4.73 kB / 1.92 kB (gzip)
async-api.30bd804e.min.js 2.92 kB / 1.46 kB (gzip)
session-manager.2a64278a.min.js 1.47 kB / 740 B (gzip)
lazy-feature-loader.0ba331d7.min.js 1.17 kB / 499 B (gzip)
Other Polyfill CDN Assets

Released Assets

Asset Name Asset Size
recorder.1b18459f-es5.min.js 166.76 kB / 52.35 kB (gzip)
nr-polyfills.min.js 52.15 kB / 17.95 kB (gzip)
session_trace-aggregate.c6e2d80e-es5.min.js 32.26 kB / 8.5 kB (gzip)
compressor.79fb47d9-es5.min.js 30.02 kB / 11.29 kB (gzip)
spa-aggregate.cf7614a6-es5.min.js 24.02 kB / 8.11 kB (gzip)
session_replay-aggregate.307e7ec3-es5.min.js 19.24 kB / 6.52 kB (gzip)
page_view_timing-aggregate.4cf1986a-es5.min.js 18.67 kB / 6.17 kB (gzip)
173.4c1277b2-es5.min.js 17.69 kB / 6.21 kB (gzip)
page_view_event-aggregate.7a2d4522-es5.min.js 12.64 kB / 4.8 kB (gzip)
jserrors-aggregate.46877c05-es5.min.js 12.38 kB / 4.42 kB (gzip)
385.1f3686e5-es5.min.js 10.62 kB / 4.16 kB (gzip)
ajax-aggregate.34fb41fa-es5.min.js 10.58 kB / 3.78 kB (gzip)
metrics-aggregate.9be0ba0b-es5.min.js 9.91 kB / 3.35 kB (gzip)
page_action-aggregate.710c9b26-es5.min.js 7.33 kB / 2.64 kB (gzip)
async-api.2b4763c8-es5.min.js 4.16 kB / 1.99 kB (gzip)
session-manager.ae4f9cf3-es5.min.js 1.7 kB / 785 B (gzip)
lazy-feature-loader.60c98be9-es5.min.js 1.2 kB / 509 B (gzip)

Built Assets

Asset Name Asset Size
recorder.1b18459f-es5.min.js 166.76 kB / 52.35 kB (gzip)
nr-polyfills.min.js 52.15 kB / 17.95 kB (gzip)
session_trace-aggregate.c836d31f-es5.min.js 32.26 kB / 8.5 kB (gzip)
compressor.79fb47d9-es5.min.js 30.02 kB / 11.29 kB (gzip)
spa-aggregate.9c283051-es5.min.js 24.02 kB / 8.11 kB (gzip)
page_view_timing-aggregate.79543323-es5.min.js 18.67 kB / 6.17 kB (gzip)
session_replay-aggregate.f4bdc759-es5.min.js 18.57 kB / 6.35 kB (gzip)
173.4c1277b2-es5.min.js 17.69 kB / 6.21 kB (gzip)
page_view_event-aggregate.f50d77d0-es5.min.js 12.64 kB / 4.8 kB (gzip)
jserrors-aggregate.d9f1a3f3-es5.min.js 12.38 kB / 4.42 kB (gzip)
ajax-aggregate.0812b00e-es5.min.js 10.58 kB / 3.78 kB (gzip)
646.abc32154-es5.min.js 10.41 kB / 4.1 kB (gzip)
metrics-aggregate.5092580c-es5.min.js 9.91 kB / 3.35 kB (gzip)
page_action-aggregate.b73fecd7-es5.min.js 7.33 kB / 2.64 kB (gzip)
async-api.2b4763c8-es5.min.js 4.16 kB / 1.99 kB (gzip)
session-manager.ae4f9cf3-es5.min.js 1.7 kB / 785 B (gzip)
lazy-feature-loader.fe02c936-es5.min.js 1.2 kB / 509 B (gzip)

@github-actions
Copy link

github-actions bot commented Jul 13, 2023

Static Badge

Last ran on July 13, 2023 17:58:00 CDT
Checking merge of (93ec48c) into main (8a6f319)

@patrickhousley patrickhousley force-pushed the pr-checks-updates branch 4 times, most recently from a50ac82 to d0a98ec Compare July 13, 2023 20:42
Copy link
Member

@metal-messiah metal-messiah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice job with this! -- Approved, just don't forget to update the PR description

@patrickhousley patrickhousley merged commit 8d0a5ff into main Jul 14, 2023
47 checks passed
@patrickhousley patrickhousley deleted the pr-checks-updates branch July 14, 2023 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants