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

lint swingset-xsnap-supervisor imports #10885

Merged
merged 2 commits into from
Jan 24, 2025
Merged

Conversation

turadg
Copy link
Member

@turadg turadg commented Jan 23, 2025

incidental

Description

In #10878 I noticed,

This missing dependency should have been discovered by our linter, and it’s not clear how it was not.

The package had disabled import/no-extraneous-dependencies for some reason. That was carried forward in the recent v9 upgrade: https://github.com/Agoric/agoric-sdk/pull/10846/files#diff-9ab4ae5d071b94a2d5dd874ceb3b9a1e85913fe7ee92e4956cb9801e59777002

This removes the special case and declares the deps so the rule passes.

Security Considerations

none

Scaling Considerations

none

Documentation Considerations

none

Testing Considerations

CI

Upgrade Considerations

none

@turadg turadg requested review from kriskowal and iomekam January 23, 2025 22:30
@turadg turadg requested a review from a team as a code owner January 23, 2025 22:30
Copy link

cloudflare-workers-and-pages bot commented Jan 23, 2025

Deploying agoric-sdk with  Cloudflare Pages  Cloudflare Pages

Latest commit: f444785
Status: ✅  Deploy successful!
Preview URL: https://b1b0fc54.agoric-sdk.pages.dev
Branch Preview URL: https://ta-lint-xsnap-supervisor.agoric-sdk.pages.dev

View logs

@turadg turadg added automerge:rebase Automatically rebase updates, then merge bypass:integration Prevent integration tests from running on PR labels Jan 24, 2025
Copy link
Contributor

@iomekam iomekam left a comment

Choose a reason for hiding this comment

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

LGTM!

@turadg turadg force-pushed the ta/lint-xsnap-supervisor branch from 32cefca to f444785 Compare January 24, 2025 23:10
@mergify mergify bot merged commit f0ccde6 into master Jan 24, 2025
76 of 77 checks passed
@mergify mergify bot deleted the ta/lint-xsnap-supervisor branch January 24, 2025 23:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge:rebase Automatically rebase updates, then merge bypass:integration Prevent integration tests from running on PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants