Repository navigation
chore: land the cleanup campaign on main - #21
Merged
Merged
Conversation
* test: type the contract suite against local source The suite loads dist at run time but had no types for it, so it was typed only once the package was built. CI lints before any build, so the unbuilt copy typed every call as any. The module is now typed from src/index, and test/tsconfig.json gives the suite the node and mocha types. * test: resolve the contract suite's types from src through a package import The suite now requires #built, a package import whose types condition points at src/index.ts and whose default points at dist/index.js. TypeScript types the module from source with no dist on disk, and Node still runs the built file. This replaces the JSDoc type on the require, which left the value any until dist existed.
* build(deps): bump @antelopejs/tooling-configs to 0.2.17 * refactor(tests): clear the type-aware findings the 0.2.17 rules report * chore: delete the emptied guardrails ratchet * chore: sync the guardrails pack after the ratchet deletion * Revert "chore: sync the guardrails pack after the ratchet deletion" This reverts commit ea96ad9. * Revert "chore: delete the emptied guardrails ratchet" This reverts commit a845d7b. * refactor(tests): keep the deliberately invalid interval as an assertion * chore: lower the ratchet to the one row that stays
…ead of a cast (#14) * test: assert the invalid interval rejection with ts-expect-error instead of a cast The conformance case for an interval outside the vocabulary now passes the invalid value directly, with a @ts-expect-error that names the rejection under test, and a Partial<SubscriptionRequest> return type so the error stays on that one line. The ratchet has no debt left, so its file is deleted; the suppressions budget raises tsExpectError from 0 to 1 for this one directive. * chore: sync the guardrails pack after the ratchet is deleted
* feat(skills): add the verify-interface verification skill * fix(skills): cover the error classes and load the conformance helpers in the verify-interface skill * fix(skills): seed breaks that build and a format door that corrupts every vocabulary
* docs: pass the interface-subscriptions docs to the docs standard * docs: turn on the docs standard * docs: correct the fallback vendorData and Stripe conformance claims in the capability matrix
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Lands
campaign/integrationat f54532c onmain. Every change on that branch landed through a verified pull request. The tree of this pull request equalscampaign/integration; the merge ofmainonly joins the histories, sincecampaign/integrationalready contains every change ofmainup to f7a522b. No package is published by this merge.