Skip to content

Conversation

@x3c41a
Copy link
Contributor

@x3c41a x3c41a commented Nov 21, 2025

just authorize-and-store pjs - to run e2e authorize_and_store test with PJS,
just authorize-and-store papi or just authorize-and-store - to test with PAPI

Relates to #108


# Test the complete workflow (builds, starts services, runs example, shuts down services)
# Parameters: api=[papi|pjs] - choose between Polkadot API (papi) or Polkadot JS (pjs)
authorize-and-store api="papi": build npm-install
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think if we want to use this command in README we'd have to refactor that README significantly

kill $IPFS_PID $ZOMBIENET_PID $RECONNECT_PID 2>/dev/null || true


exit $EXAMPLE_EXIT
Copy link
Contributor Author

Choose a reason for hiding this comment

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

jfyi: newline is not needed here. I cross-checked with ahm-dryrun repo

@x3c41a x3c41a requested a review from bkontur November 21, 2025 10:42
Copy link
Contributor

@raymondkfcheung raymondkfcheung left a comment

Choose a reason for hiding this comment

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

Please check to use $(ls zombienet-*-*) instead of hardcoded.

@x3c41a
Copy link
Contributor Author

x3c41a commented Nov 21, 2025

good catch, I had this comment in my previous PR

@x3c41a
Copy link
Contributor Author

x3c41a commented Nov 21, 2025

Please check to use $(ls zombienet-*-*) instead of hardcoded.

fixed

@x3c41a x3c41a enabled auto-merge (squash) November 27, 2025 14:17
@x3c41a x3c41a merged commit 93c7fdd into main Nov 27, 2025
7 checks passed
@x3c41a x3c41a deleted the new_just branch November 27, 2025 14:58
@@ -0,0 +1,112 @@
# Polkadot Bulletin Chain - Just Commands
Copy link
Collaborator

Choose a reason for hiding this comment

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

@x3c41a I am missing here some README.md update how to run/use this justfile

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah, I thought the description would be enough.
Generally it's is as simple as just <command>, e.g. just authorize-and-store.

let me make a PR really quick...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

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.

4 participants