Skip to content

fix(evasive-transforms,ses): fix lint warnings#3240

Open
erights wants to merge 2 commits into
masterfrom
markm-fix-lint-warnings
Open

fix(evasive-transforms,ses): fix lint warnings#3240
erights wants to merge 2 commits into
masterfrom
markm-fix-lint-warnings

Conversation

@erights

@erights erights commented May 5, 2026

Copy link
Copy Markdown
Contributor

Closes: #XXXX
Refs: #XXXX

Description

I was getting two lint warnings from running yarn lint locally:

/Users/markmiller/src/ongithub/endojs/endo/packages/evasive-transform/src/index.js
  25:1  warning  Expected JSDoc block to be aligned  jsdoc/check-alignment

/Users/markmiller/src/ongithub/endojs/endo/packages/ses/src/compartment.js
  340:3  warning  Missing JSDoc @param "args" declaration  jsdoc/require-param

The first was trivial. The second was caused by an accidental split of a doc-comment into two. Repairing that, the missing args declaration had an incorrect "...". Fixing both of those, local yarn lint is now clean.

Security Considerations

none

Scaling Considerations

none

Documentation Considerations

none

Testing Considerations

none

Compatibility Considerations

none. That is, unless something outside the endo repo relied on the mistakenly hidden type decl being absent. Unlikely.

Upgrade Considerations

none.

@erights erights self-assigned this May 5, 2026
@changeset-bot

changeset-bot Bot commented May 5, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: eaacd4f

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 28 packages
Name Type
@endo/evasive-transform Patch
ses Patch
@endo/bundle-source Patch
@endo/compartment-mapper Patch
@endo/import-bundle Patch
@endo/captp Patch
@endo/check-bundle Patch
@endo/common Patch
@endo/daemon Patch
@endo/errors Patch
@endo/exo Patch
@endo/far Patch
@endo/goblin-chat Patch
@endo/lockdown Patch
@endo/marshal Patch
@endo/module-source Patch
@endo/ocapn Patch
@endo/pass-style Patch
@endo/patterns Patch
@endo/ses-ava Patch
@endo/stream-types-test Patch
@endo/stream Patch
@endo/test262-runner Patch
@endo/cli Patch
@endo/lp32 Patch
@endo/stream-node Patch
@endo/init Patch
@endo/netstring Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

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.

1 participant