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

clean up, release and use #169

Merged
merged 13 commits into from
Aug 27, 2024
Merged

clean up, release and use #169

merged 13 commits into from
Aug 27, 2024

Conversation

turadg
Copy link
Member

@turadg turadg commented Aug 27, 2024

Adds getInstanceBoardId to support some a3p-integration use cases. cc @Chris-Hibbert

Also bumps deps for hygiene, cuts a 1.1.1 release and DRYs some proposal code to use that release.

an export was missing from 0.1.1 so I also cut 1.2.0, which is a more correct version number since it adds a feature.

Validated in Agoric/agoric-sdk#9977

@turadg turadg marked this pull request as ready for review August 27, 2024 15:57
@turadg turadg requested review from dckc and Chris-Hibbert August 27, 2024 15:59
@turadg turadg changed the title clean up, release 0.1.1 and use it clean up, release and use Aug 27, 2024
Copy link
Member

@dckc dckc left a comment

Choose a reason for hiding this comment

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

very nice

"test": "NODE_OPTIONS='--loader=tsx --no-warnings' ava"
"test": "ava"
Copy link
Member

Choose a reason for hiding this comment

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

👏 less cruft

import {
agd,
agops,
agoric,
} from '@agoric/synthetic-chain';
import { agd, agops, agoric } from '@agoric/synthetic-chain';
Copy link
Member

Choose a reason for hiding this comment

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

👏 more concise style

@@ -0,0 +1,101 @@
# Snapshot report for `test/test-exports.ts`
Copy link
Member

Choose a reason for hiding this comment

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

👏 ooh... nifty idea!

@@ -1,151 +0,0 @@
// @ts-check
Copy link
Member

Choose a reason for hiding this comment

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

👏 whee! bye-bye cruft.

@turadg turadg merged commit 643068f into main Aug 27, 2024
2 checks passed
@turadg turadg deleted the ta/cleanup branch August 27, 2024 21:04
/**
*
* @param {string} instanceName
* @returns {string | null} boardId of the named instance in agoricNames
Copy link
Member Author

Choose a reason for hiding this comment

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

This is a bug. I don't know why it passed CI. Will revisit

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.

2 participants