Skip to content

Repo filtering #48

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Repo filtering #48

wants to merge 2 commits into from

Conversation

Chadyshack
Copy link

Adds flags --archived and --forked to each of the main files.

Users can specify Yes or No to include forked or archived repositories in the results.

@CLAassistant
Copy link

CLAassistant commented Oct 29, 2024

CLA assistant check
All committers have signed the CLA.

@Chadyshack
Copy link
Author

Closes #47

@radhi1991 radhi1991 requested review from sai100 and dinpraka and removed request for sai100 November 8, 2024 05:26
@radhi1991
Copy link
Contributor

@dinpraka and @sai100 can you please check on this

@radhi1991 radhi1991 requested review from jansiraj and removed request for dinpraka February 5, 2025 09:47
@radhi1991 radhi1991 linked an issue Feb 5, 2025 that may be closed by this pull request
@radhi1991
Copy link
Contributor

@jansiraj @sai100

Copy link
Collaborator

@jansiraj jansiraj left a comment

Choose a reason for hiding this comment

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

Coding is fine but Functionality seems to be not working while testing

@@ -782,6 +782,28 @@ def arg_parser():
help="Pass the repo name list as comma separated string",
)

argparser.add_argument(
"-a",
"--archived",
Copy link
Collaborator

Choose a reason for hiding this comment

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

please update the argument parser as --include-archived or -ia and for forked --include-forked or -if for better user understanding or also change help message like "Pass Yes or No to include archived repositories for search. Default is Yes". change in all the respective files wherever used

response = requests.get(
self._base_url,
params={
"q": f"{search_query} {additional_qualifiers}",
"q": q_string,
Copy link
Collaborator

Choose a reason for hiding this comment

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

search api Not filtering archived repos correctly.Need to fix it

response = requests.get(
self._base_url,
params={
"q": f"{search_query}",
"q": q_string,
Copy link
Collaborator

Choose a reason for hiding this comment

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

search api Not filtering archived repos correctly.Need to fix it

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.

Repository Filtering
4 participants