Skip to content
This repository was archived by the owner on Mar 11, 2025. It is now read-only.

Fix Memo program ESM exports #3507

Merged
merged 8 commits into from
Aug 22, 2022
Merged

Fix Memo program ESM exports #3507

merged 8 commits into from
Aug 22, 2022

Conversation

jordaaash
Copy link
Contributor

This is the first of two PRs to address #3502, this one for the Memo program JS (which the Token program JS depends on). This fixes the issues that build systems have with .mjs files by exporting .js files. It also restructures the project to be similar to Wallet Adapter / the TypeScript monorepo template I've been using. I'll update Token in parallel.

@jordaaash jordaaash requested a review from joncinque August 22, 2022 20:02
Copy link
Contributor

@joncinque joncinque left a comment

Choose a reason for hiding this comment

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

I don't know a ton about the TypeScript compiler, so I'm just trusting a lot of the configs there. Otherwise, looks good to me! Why is there no lockfile at all now?

@jordaaash
Copy link
Contributor Author

Why is there no lockfile at all now?

Fixed!

@jordaaash jordaaash merged commit 48a859b into master Aug 22, 2022
@jordaaash jordaaash deleted the fix-esm-memo branch August 22, 2022 22:57
@jordaaash
Copy link
Contributor Author

Published in @solana/[email protected]

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants