Skip to content

Update README.md#1

Open
linfraze wants to merge 1 commit intofrazieje:mainfrom
linfraze:patch-1
Open

Update README.md#1
linfraze wants to merge 1 commit intofrazieje:mainfrom
linfraze:patch-1

Conversation

@linfraze
Copy link
Copy Markdown

No description provided.

Comment thread README.md
- Calculate your own Pi data file with [y-cruncher](http://www.numberworld.org/y-cruncher/)
- Perform the following steps:

1. Set the location of the file as an environment variable:
Copy link
Copy Markdown
Owner

@frazieje frazieje Apr 29, 2024

Choose a reason for hiding this comment

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

I like the update to use an unordered list for the data file options, but this line about setting the location of the file starts the next section, which should be titled "Configuring the data file" or "using your data file" or something like that

Comment thread README.md

### Usage
To use the HTTP API once running:
After you run the program, use the HTTP API:
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

"After you run the program, use the HTTP to search for a number in Pi:"

Should we also mention that they need to update the searchText to whatever number they want to search for?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Yes

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