Skip to content

Update README.md - #9

Merged
jvmvik merged 2 commits into
serpapi:masterfrom
contributor860:patch-1
Nov 5, 2026
Merged

jvmvik merged 2 commits into
serpapi:masterfrom
contributor860:patch-1

Conversation

@contributor860

Copy link
Copy Markdown
Contributor

Corrected grammatical and spelling errors

contributor860 and others added 2 commits November 2, 2026 20:30
Corrected grammatical and spelling errors
The template no longer adds anything over README.md, and keeping both
meant direct README edits (like serpapi#9) were silently reverted by
`make readme`. Remove the template and the `readme` make target, and
tell contributors to update README.md snippets alongside examples.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

@jvmvik jvmvik left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks! Removing README.md.erb and the make readme target matches how we maintain README.md now. The CI failures are only 'Invalid API key' errors, because forked PRs don't get the SERPAPI_KEY secret; they aren't related to this change.

@jvmvik
jvmvik merged commit 30dc2ac into serpapi:master Nov 5, 2026
1 check failed
@contributor860

Copy link
Copy Markdown
Contributor Author

Thanks for merging this PR. Will try to contribute more in the future

jvmvik added a commit that referenced this pull request Nov 5, 2026
Resolve conflicts from master's README rework and the README.md.erb removal (#9):
- drop README.md.erb (deleted on master)
- take master's README.md and re-add the Image API section under the Google Lens example
- list the Image API in the Features section

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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