Skip to content

Speed up CI, remove unused built files and node_modules, add projectId as distinct_id - #10

Merged
alex-noel merged 2 commits into
masterfrom
rebuild-action
May 18, 2026
Merged

alex-noel merged 2 commits into
masterfrom
rebuild-action

Conversation

@alex-noel

@alex-noel alex-noel commented May 15, 2026 •

Copy link
Copy Markdown
Contributor

Minor changes for posthog analytics to make sure we get them right:

  1. Use api_key instead of token as suggested by official docs (initially used the wrong one, they have outdated first example which i looked at initially)
  2. Set distinct_id to projectId just for cleaner data
  3. Increase timeout from 2000 -> 5000 just in case
    Some CI/CD and chores which don't impact the action in any way (we already use built index.js on master):
  4. Remove node_modules to reduce the diff
  5. Point to index.js instead of main.js in package.json (not affecting anything)
  6. Remove older build files, everything is in index.js now
  7. Remove unneeded cache step (the install step is already cached by default)

@alex-noel alex-noel self-assigned this May 15, 2026
@alex-noel alex-noel changed the title rebuild the posthog, add a distinct id as projectId Speed up CI, remove unused built files and node_modules, add projectId as distinct_id May 15, 2026
@alex-noel
alex-noel requested review from Copilot and sampullman and removed request for sampullman May 15, 2026 13:07

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

@alex-noel
alex-noel merged commit 4c58a9e into master May 18, 2026
1 check passed
@alex-noel
alex-noel deleted the rebuild-action branch May 18, 2026 14:50
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.

3 participants