[CI] - NPM Dependency Caching and Lighthouse Audit Action

[adithyaakrishna]

Description

This PR fixes #2773

Notes for Reviewers

• Added NPM Caching Capabilities - This is working as of now as I had worked on a similar issue for my personal portfolio repositiory. Here's the deployment script, https://github.com/adithyaakrishna/adithyaakrishna.github.io/blob/source/.github/workflows/deploy.yml and in this the build time has been reduced from 2-3mins to 1.5min. This successfully caches and uses it in the action when it's triggered.

Ref: https://github.com/adithyaakrishna/adithyaakrishna.github.io/actions - Only the two most recent action triggers

• Added Lighthouse Audit GitHub Action - WIP

Signed commits

• Yes, I signed my commits.

[adithyaakrishna]

@Nikhil-Ladha is there any way using which we can share outputs from other workflows? I am a bit stuck on that here 😶

CC @Chadha93

[Nikhil-Ladha]

I am not sure, but what output are you referring to here?

[adithyaakrishna]

@Nikhil-Ladha Is there any way where we can extract the l5io bot output which comments the preview URL for the workflow here?

[Nikhil-Ladha]

You can use the find-comment action, only if you have the PR number for which the workflow is running.
Or, if that's not possible then another way is to save the deploy URL as an artifact in the preview-site workflow and then download that artifact here and get the URL from it.

If the first one is feasible, then that is preferred.

[l5io]

🚀 Preview for commit 27a1d1a at: https://62c34e83617947399fca538d--layer5.netlify.app

[leecalcote]

Oh! This is exciting. 😃

[Nikhil-Ladha]

I was thinking about this PR, and wondered is it actually beneficial to have?
For us dependency installation hardly takes a minute, it is the site build that takes most of the time. And even after caching the npm dependencies that are not going to reduce.

We need an upgrade to v4, which has been pending a long time on me and if someone can take that up, that should help us make things faster (if not much). Because, the limitation here is GitHub actions env, which is very basic and no matter the big improvement locally, when it comes to GitHub actions the performance improvement becomes negligible.

[adithyaakrishna]

@Nikhil-Ladha I can take this up as I had worked on the v2 to v3 thing last year(#1767), but this week might be a bit difficult as our team at Red Hat is on an office trip and I am in Pune as of now. I can take this up the following one if that would be fine

CC @leecalcote

[Nikhil-Ladha]

Sure, you can update us regarding it next week 👍🏼

[l5io]

🚀 Preview for commit 6112b01 at: https://62e84a592f372b3635635b9f--layer5.netlify.app

[l5io]

🚀 Preview for commit 96f5f51 at: https://62e8ec715ed8aa2fd73cac22--layer5.netlify.app

[l5io]

🚀 Preview for commit 62418fb at: https://62ea00c04c8fb35442ba88b9--layer5.netlify.app

[Chadha93]

Lots's of great stuff here by @adithyaakrishna 💪. Good to go @Nikhil-Ladha @adithyaakrishna?

[Nikhil-Ladha]

Also, the docker.png file that you have added is not needed

[Nikhil-Ladha]

It is enabled by default in gatsbyv4. Not needed

[Nikhil-Ladha]

Lots's of great stuff here by @adithyaakrishna 💪. Good to go @Nikhil-Ladha @adithyaakrishna?

I doubt, the lighthouse.yml workflow is still incomplete

[adithyaakrishna]

@Nikhil-Ladha Yes, this is not yet complete, I am trying to work on a solution to find the preview links every time a code is pushed and this workflow is something we use at Red Hat as well. So I thought to implement something similar here

Ref: https://github.com/patternfly/patternfly-elements/blob/main/.github/workflows/visual-regression.yml#L65

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[github-actions]

Checking in... it has been awhile since we've heard from you on this issue. Are you still working on it? Please let us know and please don't hesitate to contact a MeshMate or any other community member for assistance.

---

        Be sure to join the community, if you haven't yet and please leave a ⭐ star on the project 😄

[Nikhil-Ladha]

Can we expect an update here @adithyaakrishna ?

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[stale]

This issue is being automatically closed due to inactivity. However, you may choose to reopen this issue.