8000 slug optimizations - npm edition by happythenewsad · Pull Request #12810 · empirical-org/Empirical-Core · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

slug optimizations - npm edition #12810

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

Draft
wants to merge 2 commits into
base: develop
Choose a base branch
from
Draft

Conversation

happythenewsad
Copy link
Contributor

WHAT

WHY

HOW

Screenshots

(If applicable. Also, please censor any sensitive data)

Notion Card Links

(Please provide links to any relevant Notion card(s) relevant to this PR.)

What have you done to QA this feature?

(Provide enough detail that a reviewer could assess whether additional QA should be done. For larger projects, additionally use the Engineer Feature Testing Notion template. Review Guidelines if needed: https://www.notion.so/quill/Github-PR-QA-Guidelines-49e99fc965654ceeb8c6249bd9d181d7)

PR Checklist Your Answer
Have you added and/or updated tests? (The answer should mostly be 'YES'. If you answer 'NO', please justify.)
Have you deployed to Staging? (Possible answers: YES, Not yet - deploying now!, NO - non-app change, NO - tiny change)
Self-Review: Have you done an initial self-review of the code below on Github?

@happythenewsad happythenewsad changed the title Remove devdep slug optimizations - npm edition Feb 24, 2025
Peter Kong added 2 commits March 19, 2025 11:19
update package lock

move babel stuff to devDeps, run npm install in client/

use npm ci

Purge cache

new postinstall script, remove babel override

Purge cache

update script

add --production to npm ci

removing devDependencies, temp

use --include flag in postinstall.sh

Revert "removing devDependencies, temp"

This reverts commit 48930d1.

use --omit flag instead

try NPM_CONFIG_PRODUCTION

purge cache

purge cache

another attempt

update prune command

update prune directive

revert to npm install on staging

add logging to postinstall

remove postbuild script

testing client/ postinstall script

workspaces

rewire legacy node_modules references to accomodate npm workspaces
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
55BD
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant
0