这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@kantagara
Copy link
Contributor

@kantagara kantagara commented Aug 13, 2025

closes #1292

@rob1997 rob1997 added the ready-to-merge Ready to Merge PR - this'll trigger required checks label Aug 13, 2025
@rob1997
Copy link
Collaborator

rob1997 commented Aug 13, 2025

@kantagara nice! only thing I would add is, since we don't need it anymore removing the marketplace package entirely, including the samples, and any references for CI; removing the dlls and paths in src/Setup, dependencies.json and packages.json

@rob1997 rob1997 added the documented Apply on Pull Request after documentation to allow merge label Aug 13, 2025
@kantagara
Copy link
Contributor Author

Good catch Rob!

Copy link
Collaborator

@rob1997 rob1997 left a comment

Choose a reason for hiding this comment

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

🔥

@rob1997 rob1997 merged commit b3b6967 into ChainSafe:main Aug 13, 2025
7 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documented Apply on Pull Request after documentation to allow merge ready-to-merge Ready to Merge PR - this'll trigger required checks

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove Dashboard Dependency

2 participants