search

turbot / steampipe-plugin-github

Use SQL to instantly query repositories, users, gists and more from GitHub. Open source CLI. No DB required.
https://hub.steampipe.io/plugins/turbot/github
Apache License 2.0
74 stars 29 forks source link

Fixed table github_workflow fails with error when we pass all repositories Closes #388 #400

Closed ParthaI closed 8 months ago

ParthaI commented 8 months ago

Example query results

Results ``` > select * from github_workflow where repository_full_name in (select name_with_owner from github_my_repository) +-------------------------------------------------+--------------------------------------------------------------------------------------------------------------------------------+----------+------------------------------------------------------------------------------> | repository_full_name | name | id | path > +-------------------------------------------------+--------------------------------------------------------------------------------------------------------------------------------+----------+------------------------------------------------------------------------------> | turbot/flowpipe-mod-vault | Sync Labels | 79344531 | .github/workflows/sync-labels.yml > | | | | > | | | | > | | | | > | | | | > | | | | > | | | | > | | | | > | | | | > | turbot/flowpipe-mod-vault | Stale Issues and PRs | 29393403 | .github/workflows/stale.yml > | | | | > | | | | > | | | | > | | | | > | | | | > | | | | > | | | | > | | | | > | | | | > | | | | > | | | | > | | | | > | | | | > | | | | > | | | | > | | | | > | turbot/steampipe-mod-azure-insights | Sync Labels | 32812937 | .github/workflows/sync-labels.yml > | | | | > | | | | > | | | | > | | | | > | | | | > | | | | > | | | | > | | | | > | turbot/steampipe-mod-azure-insights | Stale Issues and PRs | 91238345 | .github/workflows/stale.yml > | | | | > | | | | > | | | | > | | | | > | | | | > | | | | > | | | | > | | | | > | | | | > | | | | > | | | | > | | | | > | | | | > | | | | > | | | | > | | | | > | turbot/flowpipe-mod-openai | Stale Issues and PRs | 43024328 | .github/workflows/stale.yml > | | | | > | | | | > | | | | > | | | | > | | | | > | | | | > | | | | > | | | | > | | | | > | | | | > | | | | > | | | | > | | | | > | | | | > | | | | > | | | | > | turbot/flowpipe-mod-openai | Sync Labels | 47382910 | .github/workflows/sync-labels.yml > Time: 303.5s. Rows fetched: 370. Hydrate calls: 0. ```
misraved commented 8 months ago

Thanks @ParthaI for the fix, does the change in the package affect other tables?

ParthaI commented 8 months ago

Thanks @ParthaI for the fix, does the change in the package affect other tables?

@misraved, I don't think the modification in the package will impact any other tables. After querying 1-2 other tables, everything seems to be working fine.