Skip to content
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

Remove inconsistent terms for tracking scope #1087

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

amirhhashemi
Copy link
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.

Copy link

stackblitz bot commented Feb 16, 2025

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
Copy link

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
Copy link

netlify bot commented Feb 16, 2025

Deploy Preview for solid-docs ready!

Name Link
🔨 Latest commit c2b6037
🔍 Latest deploy log https://app.netlify.com/sites/solid-docs/deploys/67b5087e07e9af0009e94b34
😎 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
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.

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.

2 participants