Skip to content

Conversation

@drew-richardson
Copy link
Contributor

Description

Enhance the composableschemadsl/compiler.Compile API by adding a new option SourceFS which instead of reading from the filesystem, reads from a virtual go filesystem. This makes no changes to the zed command line, but opens up the possibility to add something like zed compile --using-archive schemapackage.zip root.zed in the future.

Testing

I ran the unit tests and verified they passed, and I also added a trivial test for SourceFS.

References

#2807

@drew-richardson drew-richardson requested a review from a team as a code owner January 12, 2026 16:22
@github-actions github-actions bot added the area/tooling Affects the dev or user toolchain (e.g. tests, ci, build tools) label Jan 12, 2026
@github-actions
Copy link

github-actions bot commented Jan 12, 2026

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

Copy link
Contributor

@tstirrat15 tstirrat15 left a comment

Choose a reason for hiding this comment

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

This looks good to me.

@codecov
Copy link

codecov bot commented Jan 12, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.50%. Comparing base (9300588) to head (34d864e).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2808      +/-   ##
==========================================
- Coverage   78.52%   78.50%   -0.02%     
==========================================
  Files         459      459              
  Lines       44445    44448       +3     
==========================================
- Hits        34896    34889       -7     
- Misses       6723     6730       +7     
- Partials     2826     2829       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@drew-richardson
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

authzedbot added a commit to authzed/cla that referenced this pull request Jan 12, 2026
@tstirrat15 tstirrat15 enabled auto-merge January 12, 2026 16:48
@tstirrat15 tstirrat15 added this pull request to the merge queue Jan 12, 2026
Merged via the queue into authzed:main with commit 9171c3f Jan 12, 2026
45 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Jan 12, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area/tooling Affects the dev or user toolchain (e.g. tests, ci, build tools)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants