Skip to content

Commit

Permalink
✨ use npm cache and ci install, better chunking
Browse files Browse the repository at this point in the history
  • Loading branch information
actuallymentor committed Apr 3, 2024
1 parent 3a457e0 commit fa7644b
Show file tree
Hide file tree
Showing 16 changed files with 276 additions and 342 deletions.
7 changes: 4 additions & 3 deletions .github/workflows/deploy-changes.yml
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ jobs:
cache: "npm"
- name: Install dependencies
if: steps.changed-firebase-files.outputs.any_changed == 'true'
run: npm i
run: npm ci

#########
# Deploy
Expand Down Expand Up @@ -85,10 +85,11 @@ jobs:
if: steps.changed-functions-files.outputs.any_changed == 'true'
with:
node-version-file: "functions/.nvmrc"
cache-dependency-path: "functions/package-lock.json"
cache: "npm"
- name: Install functions dependencies
if: steps.changed-functions-files.outputs.any_changed == 'true'
run: npm i
run: npm ci
working-directory: functions

#####################
Expand Down Expand Up @@ -151,7 +152,7 @@ jobs:
cache: "npm"
- name: Install dependencies
if: steps.changed-frontend-files.outputs.any_changed == 'true'
run: npm i
run: npm ci

# Set up environment
- name: Create PROD environment files
Expand Down
13 changes: 0 additions & 13 deletions build/asset-manifest.json

This file was deleted.

36 changes: 35 additions & 1 deletion build/index.html
Original file line number Diff line number Diff line change
@@ -1 +1,35 @@
<!doctype html><html lang="en"><head><meta charset="utf-8"/><link rel="icon" href="/favicon.ico"/><meta name="viewport" content="width=device-width,initial-scale=1"/><meta name="theme-color" content="#000000"/><meta name="description" content="Web site created using create-react-app"/><link rel="apple-touch-icon" href="/logo192.png"/><link rel="manifest" href="/manifest.json"/><title>React App</title><script defer="defer" src="/static/js/main.6a95a3cf.js"></script><link href="/static/css/main.97be39ad.css" rel="stylesheet"></head><body><noscript>You need to enable JavaScript to run this app.</noscript><div id="root"></div></body></html>
<!DOCTYPE html>
<html lang="en">

<head>
<meta charset="utf-8" />
<link rel="icon" href="/favicon.ico" />
<meta name="viewport" content="width=device-width, initial-scale=1" />
<meta name="theme-color" content="#000000" />
<meta name="description" content="Basic command-line line tracking" />
<link rel="apple-touch-icon" href="/logo192.png" />
<!--
manifest.json provides metadata used when your web app is installed on a
user's mobile device or desktop. See https://developers.google.com/web/fundamentals/web-app-manifest/
-->
<link rel="manifest" href="/manifest.json" />
<!--
Notice the use of in the tags above.
It will be replaced with the URL of the `public` folder during the build.
Only files inside the `public` folder can be referenced from the HTML.
Unlike "/favicon.ico" or "favicon.ico", "/favicon.ico" will
work correctly both with client-side routing and a non-root public URL.
Learn how to configure a non-root public URL by running `npm run build`.
-->
<title>Unidentified Analytics</title>
<script type="module" crossorigin src="/assets/index-wYtuajCS.js"></script>
<link rel="stylesheet" crossorigin href="/assets/index-BGUfl4AW.css">
</head>

<body>
<noscript>You need to enable JavaScript to run this app.</noscript>
<div id="root"></div>
</body>

</html>
6 changes: 3 additions & 3 deletions build/manifest.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"short_name": "React App",
"name": "Create React App Sample",
"short_name": "Unidentified Analytics",
"name": "Unidentified Analytics",
"icons": [
{
"src": "favicon.ico",
Expand All @@ -22,4 +22,4 @@
"display": "standalone",
"theme_color": "#000000",
"background_color": "#ffffff"
}
}
2 changes: 0 additions & 2 deletions build/static/css/main.97be39ad.css

This file was deleted.

1 change: 0 additions & 1 deletion build/static/css/main.97be39ad.css.map

This file was deleted.

3 changes: 0 additions & 3 deletions build/static/js/main.6a95a3cf.js

This file was deleted.

276 changes: 0 additions & 276 deletions build/static/js/main.6a95a3cf.js.LICENSE.txt

This file was deleted.

1 change: 0 additions & 1 deletion build/static/js/main.6a95a3cf.js.map

This file was deleted.

Loading

0 comments on commit fa7644b

Please sign in to comment.