Skip to content

feat(issues): add file and directory based issue filtering#106

Merged
siddhant-deepsource merged 9 commits into
DeepSourceCorp:masterfrom
burntcarrot:filtering-files
Feb 25, 2022
Merged

feat(issues): add file and directory based issue filtering#106
siddhant-deepsource merged 9 commits into
DeepSourceCorp:masterfrom
burntcarrot:filtering-files

Conversation

@burntcarrot
Copy link
Copy Markdown
Contributor

Related #98.

Adds directory based issue filtering (both single and multiple directories):

deepsource issues list command/ deepsource/ --repo github.com/deepsourcelabs/cli

File-based issue filtering is also supported (both single and multiple files):

deepsource issues list command/auth/logout/logout.go deepsource/auth/mutations/register_device.go --repo github.com/deepsourcelabs/cli

Combinations of both files and directories are also supported:

deepsource issues list command/ deepsource/auth/mutations/register_device.go --repo github.com/deepsourcelabs/cli

Signed-off-by: burntcarrot aadhav.n1@gmail.com

* add directory based issue filtering
* fix panic while fetching issues related to a file

Signed-off-by: burntcarrot <aadhav.n1@gmail.com>
Comment thread command/issues/list/list.go Outdated
Comment thread command/issues/list/list.go Outdated
Comment thread deepsource/issues/queries/list_file_issues.go
Signed-off-by: burntcarrot <aadhav.n1@gmail.com>
Comment thread command/issues/list/utils.go Outdated
Comment thread command/issues/list/list.go Outdated
Comment thread command/issues/list/list.go Outdated
* fix issue de-duplication
* refactor utilities

Signed-off-by: burntcarrot <aadhav.n1@gmail.com>
Comment thread command/issues/list/list.go Outdated
Comment thread command/issues/list/utils.go Outdated
Signed-off-by: burntcarrot <aadhav.n1@gmail.com>
Comment thread command/issues/list/list.go Outdated
Comment thread command/issues/list/utils.go Outdated
Copy link
Copy Markdown
Contributor

@siddhant-deepsource siddhant-deepsource left a comment

Choose a reason for hiding this comment

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

This looks almost ready. Have made couple of comments. PTAL. Also, would be great if you can add some tests for this.

Signed-off-by: burntcarrot <aadhav.n1@gmail.com>
@burntcarrot
Copy link
Copy Markdown
Contributor Author

Adding tests.

* add tests for issue filtering
* add dummy issue data for tests

Signed-off-by: burntcarrot <aadhav.n1@gmail.com>
Comment thread command/issues/list/list_test.go Outdated
Signed-off-by: burntcarrot <aadhav.n1@gmail.com>
@siddhant-deepsource
Copy link
Copy Markdown
Contributor

The test-coverage reporting seems to work now.

@burntcarrot burntcarrot deleted the filtering-files branch July 25, 2022 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants