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.

Sign up for GitHub

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

Jump to bottom

chore: remove Go checks #7907

Draft
wants to merge 16 commits into
base: main
Choose a base branch
from
Draft

chore: remove Go checks #7907

wants to merge 16 commits into from

Conversation

nikpivkin
Copy link
Contributor

Description

Part of #7718

Related PRs

Checklist

  • I've read the guidelines for contributing to this repository.
  • I've followed the conventions in the PR title.
  • I've added tests that prove my fix is effective or that my feature works.
  • I've updated the documentation with the relevant information (if needed).
  • I've added usage information (if the PR introduces new options)
  • I've included a "before" and "after" example to the description (if the PR is a user interface change).

@nikpivkin
refactor(test): use Rego checks in module tests
7f06548 
Signed-off-by: nikpivkin <nikita.pivkin@smartforce.io>
@nikpivkin
refactor(test): use tf scanner instead of executor
0ef7b50 
Signed-off-by: nikpivkin <nikita.pivkin@smartforce.io>
@nikpivkin
refactor(test): use Rego checks in ignore tests
60135c7 
Signed-off-by: nikpivkin <nikita.pivkin@smartforce.io>
@nikpivkin
refactor(test): reuse Rego check
ccd1ba0 
Signed-off-by: nikpivkin <nikita.pivkin@smartforce.io>
@nikpivkin
refactor(test): use Rego checks in json tests
1b8d2da 
Signed-off-by: nikpivkin <nikita.pivkin@smartforce.io>
@nikpivkin
fix tests for Go custom checks
b81097a 
Signed-off-by: nikpivkin <nikita.pivkin@smartforce.io>
@nikpivkin
refactor(test): use Rego checks in count tests
6c5cb42 
Signed-off-by: nikpivkin <nikita.pivkin@smartforce.io>
@nikpivkin
fix linter issues
362998a 
Signed-off-by: nikpivkin <nikita.pivkin@smartforce.io>
@nikpivkin
test: reuse Rego check in terraform tests
6eb5d35 
Signed-off-by: nikpivkin <nikita.pivkin@smartforce.io>
@nikpivkin
simplify testing of rules with different prefixes
9dc0fe3 
Signed-off-by: nikpivkin <nikita.pivkin@smartforce.io>
@nikpivkin
scan right target
426e221 
Signed-off-by: nikpivkin <nikita.pivkin@smartforce.io>
@nikpivkin
refactor: use table tests
61d206e 
Signed-off-by: nikpivkin <nikita.pivkin@smartforce.io>
@nikpivkin
fix for-each tests
81df7bb 
Signed-off-by: nikpivkin <nikita.pivkin@smartforce.io>
@nikpivkin
chore: remove support for custom Terraform checks
4721240 
Signed-off-by: nikpivkin <nikita.pivkin@smartforce.io>
@nikpivkin
chore: remove Go checks
29d4eb4 
Signed-off-by: nikpivkin <nikita.pivkin@smartforce.io>
@nikpivkin
remove executor pool
fff6927 
Signed-off-by: nikpivkin <nikita.pivkin@smartforce.io>
@nikpivkin nikpivkin mentioned this pull request Nov 19, 2024
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@simar7 simar7 Awaiting requested review from simar7 simar7 will be requested when the pull request is marked ready for review simar7 is a code owner

@knqyf263 knqyf263 Awaiting requested review from knqyf263 knqyf263 will be requested when the pull request is marked ready for review knqyf263 is a code owner

@DmitriyLewen DmitriyLewen Awaiting requested review from DmitriyLewen DmitriyLewen will be requested when the pull request is marked ready for review DmitriyLewen is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@nikpivkin