Skip to content

Conversation

PabloCastellano
Copy link
Member

@PabloCastellano PabloCastellano commented May 10, 2023

@PabloCastellano PabloCastellano force-pushed the up-docs branch 4 times, most recently from b9e7f72 to 8805f5f Compare May 10, 2023 10:40
Copy link
Member

@0xGabi 0xGabi left a comment

Choose a reason for hiding this comment

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

Looks good. I left a single comment to address before we can merge.

README.md Outdated
Using environment variables

```bash
docker run -e BLOBSCAN_API_ENDPOINT=http://blobscan-api:3001 -e BEACON_NODE_RPC=http://beacon:3500 -e EXECUTION_NODE_URL=http://execution:8545 --rm blossomlabs/blobscan-indexer:master
Copy link
Member

Choose a reason for hiding this comment

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

I think we are repeating the --rm blossomlabs/blobscan-indexer:master section twice.

Copy link
Member Author

Choose a reason for hiding this comment

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

why? these two commands are equivalent ways of running the container with environment variables

Copy link
Member Author

Choose a reason for hiding this comment

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

I've added a comment for the second command. Let me know if it's clearer now!

@0xGabi 0xGabi merged commit f2e8835 into master May 13, 2023
@0xGabi 0xGabi deleted the up-docs branch May 13, 2023 14:05
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