Skip to content

refactor: remove Segment plugin references and related configurations#3189

Merged
Iru89 merged 4 commits intomasterfrom
feature-BLT-2248-get-rid-of-segment--use-mixpan
Mar 25, 2026
Merged

refactor: remove Segment plugin references and related configurations#3189
Iru89 merged 4 commits intomasterfrom
feature-BLT-2248-get-rid-of-segment--use-mixpan

Conversation

@MarcosCA
Copy link
Copy Markdown
Contributor

This commit removes all references to the Segment plugin from the documentation and configuration files, including the README, sidebar, and environment variables in GitHub workflows. The Segment plugin documentation file has also been deleted, streamlining the analytics integration options available in the project.

Description

Context

Approach taken / Explain the design

To document / Usage example

Testing

The pull request...

  • has unit tests
  • has integration tests
  • doesn't need tests because... [provide a description]

This commit removes all references to the Segment plugin from the documentation and configuration files, including the README, sidebar, and environment variables in GitHub workflows. The Segment plugin documentation file has also been deleted, streamlining the analytics integration options available in the project.
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: c8397e19ac

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines -1 to -4
---

title: Plugin Segment
id: plugin-segment
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Preserve the removed Segment doc URL with a redirect

When the docs site deploys this deletion, existing links to /docs/plugins/plugin-segment (bookmarks, blog posts, search results) will start returning 404s because the page is removed outright here and docs/_redirects does not add a replacement route. This repo already maintains redirects for retired docs URLs, so adding one for the Segment page would avoid breaking inbound links while still hiding it from the sidebar.

Useful? React with 👍 / 👎.

Copy link
Copy Markdown
Contributor

@vanbasten17 vanbasten17 left a comment

Choose a reason for hiding this comment

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

I think plugin-segment could be removed as well if still present

@vanbasten17
Copy link
Copy Markdown
Contributor

@MarcosCA check failing docs build

This commit upgrades the GitHub Actions workflows to use actions/checkout@v4 and actions/setup-node@v4, while also updating the Node.js version from 16 to 18 in both workflows and the package.json file. This ensures compatibility with the latest features and improvements in Node.js.
@pull-request-size pull-request-size bot added size/L and removed size/M labels Mar 23, 2026
Marcos Calafat added 2 commits March 23, 2026 14:25
This commit simplifies the header generation logic in the bump-readmes-to-docs script by removing unnecessary line breaks and improving readability. The structure of the header remains intact while enhancing the overall clarity of the code.
This commit updates the Node.js version in the GitHub Actions workflows and the package.json file from 18 to 22, ensuring compatibility with the latest features and improvements in Node.js.
@Iru89 Iru89 merged commit 44ec9d5 into master Mar 25, 2026
4 checks passed
@Iru89 Iru89 deleted the feature-BLT-2248-get-rid-of-segment--use-mixpan branch March 25, 2026 12:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants