Skip to content

Document constrained isolated loader lookup#3

Merged
pboling merged 4 commits into
mainfrom
docs/constrain-isolated-loader-lookup
May 21, 2026
Merged

Document constrained isolated loader lookup#3
pboling merged 4 commits into
mainfrom
docs/constrain-isolated-loader-lookup

Conversation

@pboling
Copy link
Copy Markdown
Member

@pboling pboling commented May 20, 2026

Summary

  • update zero-top-level namespace examples to use an explicit auth-sanitizer requirement
  • document a loader.rb presence check before evaluating the isolated loader

Testing

  • documentation-only change

Copilot AI review requested due to automatic review settings May 20, 2026 20:34
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the README guidance for loading auth-sanitizer without introducing new top-level namespaces, making the loader lookup more explicit and robust.

Changes:

  • Update the “Zero Top-Level Namespace Additions” examples to require auth-sanitizer ~> 0.1 and >= 0.1.3.
  • Document checking for lib/auth_sanitizer/loader.rb existence and raising a LoadError with a clearer message before attempting to evaluate it.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

uses: actions/checkout@v6

- name: Install tree-sitter library
uses: kettle-rb/ts-grammar-action@main
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This is my own action, so I pin it in my own repos as a canary.

@pboling pboling merged commit 1ae62ee into main May 21, 2026
37 of 38 checks passed
@pboling pboling deleted the docs/constrain-isolated-loader-lookup branch May 21, 2026 06:44
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.

4 participants