Skip to content

Conversation

@arjansingh
Copy link
Contributor

@arjansingh arjansingh commented Sep 1, 2025

Summary

I was reading up on settings, feature flags, stores, and services to prep for my current project this morning.

Decided to update documentation to reflect current state of the world.

Changes

What:

  • CLAUDE.md
  • docs/SETTINGS.md
  • src/services/README.md
  • src/stores/README.md

Review Focus

If you think some of this information is inaccurate please let me know.

┆Issue is synchronized with this Notion page by Unito

@arjansingh arjansingh requested a review from a team as a code owner September 1, 2025 20:26
@dosubot dosubot bot added the size:XL This PR changes 500-999 lines, ignoring generated files. label Sep 1, 2025
@github-actions
Copy link

github-actions bot commented Sep 1, 2025

🎭 Playwright Test Results

All tests passed across all browsers!

⏰ Completed at: 09/01/2025, 08:50:12 PM UTC

📊 Test Reports by Browser


🎉 Your tests are passing across all browsers!

@socket-security
Copy link

socket-security bot commented Sep 1, 2025

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedhusky@​9.0.111001005878100
Addedjsondiffpatch@​0.6.01001006980100
Addedidentity-obj-proxy@​3.0.01001007076100
Addedfuse.js@​7.0.01001009077100
Addedglob@​11.0.310010010079100
Addedglobals@​15.15.01001008487100
Addedfs-extra@​11.2.010010010086100
Addedhappy-dom@​15.11.01001008894100
Addedjiti@​2.4.29810010088100
Addedknip@​5.62.0981009290100
Addedlint-staged@​15.2.79910010091100
Addedink@​6.2.29810010094100

View full report

2. Use GitHub API for branches/PRs/metadata
3. Curl GitHub website if needed

## Settings and Feature Flags Quick Reference
Copy link
Member

Choose a reason for hiding this comment

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

is this information in scope/needed for the root CLAUDE.md?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Haha according to Claude it was, when i asked it for advice and what to add. I can follow up and just delete everything but the links to the README's @benceruleanlu. Let me know if you want that.

@arjansingh arjansingh merged commit 481e3b5 into main Sep 1, 2025
11 checks passed
@arjansingh arjansingh deleted the docs/update-readmes branch September 1, 2025 22:31
@benceruleanlu benceruleanlu mentioned this pull request Sep 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XL This PR changes 500-999 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants