Skip to content

Conversation

@tk-o
Copy link
Contributor

@tk-o tk-o commented May 28, 2025

Best to be reviewed on commit by commit basis.

This PR is a first step towards unified configuration management (parsers & invariants) for ENSNode to be used across our applications, for example, ENSAdmin, and ENSIndexer.

I've been using the following PR as an example of renaming packages:

Post-release tasks:

  • perform any cleanup that might be needed in NPM. In NPM we should ensure @ensnode/utils is no longer published to NPM but @ensnode/ensnode-sdk is

Resolves #630

tk-o added 2 commits May 28, 2025 17:05
Have a single export path (`index`) that will cover all exported memebers of the package, and also allow the granular `utils` export.
@changeset-bot
Copy link

changeset-bot bot commented May 28, 2025

🦋 Changeset detected

Latest commit: 5569870

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

This PR includes changesets to release 10 packages
Name Type
@ensnode/ensrainbow-sdk Minor
@ensnode/ensnode-sdk Minor
ensindexer Minor
ensrainbow Minor
ensadmin Minor
@ensnode/ponder-metadata Minor
@ensnode/ens-deployments Minor
@ensnode/ensnode-schema Minor
@ensnode/ponder-subgraph Minor
@ensnode/shared-configs Minor

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

@vercel
Copy link

vercel bot commented May 28, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
admin.ensnode.io ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 28, 2025 6:19pm
ensnode.io ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 28, 2025 6:19pm
ensrainbow.io ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 28, 2025 6:19pm

Copy link
Member

@lightwalker-eth lightwalker-eth left a comment

Choose a reason for hiding this comment

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

@tk-o Great to see this 👍 Shared a few small suggestions. Thanks!

Copy link
Member

@lightwalker-eth lightwalker-eth left a comment

Choose a reason for hiding this comment

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

@tk-o Looks super! 🚀 Great work 👍

@tk-o tk-o merged commit af2cb03 into main May 28, 2025
7 checks passed
@tk-o tk-o deleted the refactor/630-rename-ensnode-utils-to-ensnode-sdk branch May 28, 2025 18:20
@github-actions github-actions bot mentioned this pull request May 28, 2025
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.

Rename ensnode-utils to ensnode-sdk

3 participants