Skip to content
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

feat: analyzers command #167

Closed
wants to merge 22 commits into from
Closed

feat: analyzers command #167

wants to merge 22 commits into from

Conversation

siddhant-deepsource
Copy link
Contributor

Adds the analyzers command to the DeepSource CLI.

@cla-bot cla-bot bot added the cla-signed label Jul 8, 2022
siddhant-deepsource and others added 19 commits July 18, 2022 20:01
* chore: init deepsource analyzer init command

Signed-off-by: Siddhant N Trivedi <siddhant@deepsource.io>

* chore: delete redundant types data

Signed-off-by: Siddhant N Trivedi <siddhant@deepsource.io>

* chore: don't create the directory if it already exists

Signed-off-by: Siddhant N Trivedi <siddhant@deepsource.io>

* chore: set default value of analyzer from shortcode

Signed-off-by: Siddhant N Trivedi <siddhant@deepsource.io>

* chore: minor tweak

Signed-off-by: Siddhant N Trivedi <siddhant@deepsource.io>

* chore: fix path formatting

Signed-off-by: Siddhant N Trivedi <siddhant@deepsource.io>

* tests: add tests for initAnalyzer method

Signed-off-by: Siddhant N Trivedi <siddhant@deepsource.io>

* chore: add more test case and exit init if the analyzer already exists

Signed-off-by: Siddhant N Trivedi <siddhant@deepsource.io>

* Update init.go

* Update init.go

* chore: remove analyzer.toml directory

Signed-off-by: Siddhant N Trivedi <siddhant@deepsource.io>

* chore: use bytes.Equal for matching []bytes

Signed-off-by: Siddhant N Trivedi <siddhant@deepsource.io>

* chore: address review comments

Signed-off-by: Siddhant N Trivedi <siddhant@deepsource.io>

* chore: debug

Signed-off-by: Siddhant N Trivedi <siddhant@deepsource.io>

* chore: fix DeepSource issues and add skipcq to some

Signed-off-by: Siddhant N Trivedi <siddhant@deepsource.io>

* Format code with gofmt

Co-authored-by: deepsource-autofix[bot] <62050782+deepsource-autofix[bot]@users.noreply.github.com>
Signed-off-by: Siddhant N Trivedi <siddhant@deepsource.io>
Signed-off-by: Siddhant N Trivedi <siddhant@deepsource.io>
* feat: add issues page

* fix: merge conflicts

* feat: add occurences page

* chore: update sprites

* chore: ignore node modules

* fix: template errors

* feat: sticky layouts

* feat: add favicon

* feat: add meta data

* feat: update icons

* Delete hello.html

* chore: ignore certain files while embedding and other minor fixes

Signed-off-by: Siddhant N Trivedi <siddhant@deepsource.io>

* Autofix issues in 1 file

Resolved issues in command/analyzer/dryrun/render/route.go via DeepSource Autofix

* chore: fix DeepSource issue

Signed-off-by: Siddhant N Trivedi <siddhant@deepsource.io>

* chore: add skipcq

Signed-off-by: Siddhant N Trivedi <siddhant@deepsource.io>

Co-authored-by: Sourya Vatsyayan <sourya@deepsource.io>
Co-authored-by: Siddhant N Trivedi <siddhant@deepsource.io>
Co-authored-by: deepsource-autofix[bot] <62050782+deepsource-autofix[bot]@users.noreply.github.com>
* chore: init analyzer push command

Signed-off-by: Siddhant N Trivedi <siddhant@deepsource.io>

* chore: fix merge conflicts

Signed-off-by: Siddhant N Trivedi <siddhant@deepsource.io>

* chore: fix test failure

Signed-off-by: Siddhant N Trivedi <siddhant@deepsource.io>
This commit fixes the style issues introduced in b62e522 according to the output
from gofumpt.

Details: https://deepsource.io/gh/deepsourcelabs/cli/transform/a0b67c25-044a-4e4d-b6fc-4f6ad2a540ed/

Co-authored-by: deepsource-autofix[bot] <62050782+deepsource-autofix[bot]@users.noreply.github.com>
Signed-off-by: Siddhant N Trivedi <siddhant@deepsource.io>
@siddhant-deepsource
Copy link
Contributor Author

Closing for now. Will reopen once the work on it is resumed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants