Skip to content

Incomplete information about limitations of REST API for code search #37124

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
1 task done
brabster opened this issue Mar 27, 2025 · 6 comments
Closed
1 task done

Incomplete information about limitations of REST API for code search #37124

brabster opened this issue Mar 27, 2025 · 6 comments
Labels
content This issue or pull request belongs to the Docs Content team fix-internally Triggers a workflow to copy the issue internally and close the current issue needs SME This proposal needs review from a subject matter expert

Comments

@brabster
Copy link

brabster commented Mar 27, 2025

Code of Conduct

What article on docs.github.com is affected?

https://docs.github.com/en/rest/search/search?apiVersion=2022-11-28

What part(s) of the article would you like to see updated?

I think this documentation should clearly call out that the REST API uses a legacy search syntax, the limitations of the REST code search API as described over in the legacy docs, and that results via the API are unlikely to match the results obtained through the web interface.

I just wasted a few hours building a tool to search for untrusted GitHub actions, only to discover it appeared to work well on the first couple of orgs I tried it on, but then missing results on the third led me to find the legacy docs that explained why it wasn't working (max 2 fragments, only recently-searched repos 🤦)

There are three limitations outlined in the doc, but there are ten in the full list. Had I been directed at the full list, I think it would have been clear that it was not fit for my purpose without wasting my time.

Additional information

I can submit a PR for the main doc, but I spent most of my time in the API documentation which is where the partial list of limitations is. It looks like that section is generated and not editable?

@brabster brabster added the content This issue or pull request belongs to the Docs Content team label Mar 27, 2025
Copy link

welcome bot commented Mar 27, 2025

Thanks for opening this issue. A GitHub docs team member should be by to give feedback soon. In the meantime, please check out the contributing guidelines.

@github-actions github-actions bot added the triage Do not begin working on this issue until triaged by the team label Mar 27, 2025
@Sharra-writes
Copy link
Contributor

@brabster Thanks so much for opening an issue! It is true that some of the REST API documentation is generated and has to be fixed internally. I'll get this triaged for review, and then we'll know what the limitations are!

@Sharra-writes Sharra-writes added needs SME This proposal needs review from a subject matter expert and removed triage Do not begin working on this issue until triaged by the team labels Mar 27, 2025
Copy link
Contributor

Thanks for opening an issue! We've triaged this issue for technical review by a subject matter expert 👀

@Sharra-writes
Copy link
Contributor

@brabster This will definitely need to be looked at internally, but I haven't transferred it yet because I've been trying to find out why it wasn't already documented. I'll let you know if I get an answer, but I may end up having to send it over to the team to look into in more depth.

@Sharra-writes
Copy link
Contributor

No one seems to have answers on this, so I'm going to transfer it internally to be examined.

@Sharra-writes Sharra-writes added the fix-internally Triggers a workflow to copy the issue internally and close the current issue label Apr 10, 2025
@docs-bot
Copy link
Collaborator

Thank you for opening this issue! Updates to this documentation must be made internally. I have copied your issue to an internal issue, so I will close this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content This issue or pull request belongs to the Docs Content team fix-internally Triggers a workflow to copy the issue internally and close the current issue needs SME This proposal needs review from a subject matter expert
Projects
None yet
Development

No branches or pull requests

4 participants
@brabster @docs-bot @Sharra-writes and others