Skip to content

Remove inconsistent terms for tracking scope#1087

Merged
LadyBluenotes merged 16 commits intosolidjs:mainfrom
amirhhashemi:use-consistent-term-for-tracking-scope
Apr 21, 2025
Merged

Remove inconsistent terms for tracking scope#1087
LadyBluenotes merged 16 commits intosolidjs:mainfrom
amirhhashemi:use-consistent-term-for-tracking-scope

Conversation

@amirhhashemi
Copy link
Copy Markdown
Collaborator

  • I have read the Contribution guide
  • This PR references an issue (except for typos, broken links, or other minor problems)

Description(required)

This PR replaces all instances of "reactive context" and "reactive scope" with "tracking scope" based on WRITING.md.

@bolt-new-by-stackblitz
Copy link
Copy Markdown

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@amirhhashemi amirhhashemi changed the title Use consistent terms for tracking scope Use consistent term for tracking scope Feb 16, 2025
@netlify
Copy link
Copy Markdown

netlify bot commented Feb 16, 2025

Deploy Preview for solid-docs ready!

Name Link
🔨 Latest commit a93c45a
🔍 Latest deploy log https://app.netlify.com/sites/solid-docs/deploys/67b1fafe1f08e60008bd9393
😎 Deploy Preview https://deploy-preview-1087--solid-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@amirhhashemi amirhhashemi force-pushed the use-consistent-term-for-tracking-scope branch from 1603b9d to 20bb4f9 Compare February 16, 2025 14:51
@amirhhashemi amirhhashemi changed the title Use consistent term for tracking scope Remove inconsistent terms for tracking scope Feb 16, 2025
@netlify
Copy link
Copy Markdown

netlify bot commented Feb 16, 2025

Deploy Preview for solid-docs ready!

Name Link
🔨 Latest commit 4346945
🔍 Latest deploy log https://app.netlify.com/sites/solid-docs/deploys/6806ad724b43f2000890d4f7
😎 Deploy Preview https://deploy-preview-1087--solid-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@LadyBluenotes
Copy link
Copy Markdown
Member

This might need to be reviewed / talked about before putting into place. Admittedly, the Writing.md when it comes to the terms to use / not use needs updating.

@jer3m01
Copy link
Copy Markdown
Member

jer3m01 commented Mar 3, 2025

This might need to be reviewed / talked about before putting into place. Admittedly, the Writing.md when it comes to the terms to use / not use needs updating.

Oh my bad missed your comment before clicking approve 😬

@LadyBluenotes LadyBluenotes enabled auto-merge (squash) April 21, 2025 20:41
@LadyBluenotes LadyBluenotes merged commit 9a11242 into solidjs:main Apr 21, 2025
6 of 7 checks passed
@amirhhashemi amirhhashemi deleted the use-consistent-term-for-tracking-scope branch April 22, 2025 05:21
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