* fix: change error message to human readable

* feat: add new validateYaml command

* test: fix tests

* fix: add filename to default print

* fix: add new validate yaml context

* test: add validate yaml tests

* feat: remove validate-yaml from help menu

* feat: export logic to yamlValidator pkg

* test: add test to pkg

* fix: removed unnecessary example comment

* fix: change variable names

* fix: deleted comments

* fix: removed human readable error message

* fix: readded tests with files

* fix: change extractor error mesage

* feat: add pkg dependency

* fix: remove comments
Co-authored-by: shalev avhar <shalev@datree.io>

* feat: add rule 54 and toggle off rule 14

* test: fix junit output test for swapping enabledByDefault rules

* init

* refactor: connect policies.yaml to schema file

* style: fix formating

* test: add test case for invalid schema

* docs: add comment for clarity

* fix: update schema to latest version

* fix: remove schema validation

* test: remove test

* refactor: move validatePoliciesYaml to a folder

* fix: add defaultRulesSchema.json back

* refactor: move validatePoliciesYaml into a separate package

* refactor: fix goimports

* refactor: add json schema configuration to defaultRules.yaml for intelij

* refactor: add schema type for defaultRules.yaml

* refactor: add schema for json schemas

* test: add test case for custom rules null

* fix: allow customRules to be null

* test: add test case for missing api verison

* feat: add instance path in --policy-config error messages

* test: add more test cases

* fix: allow for policyRules to be null

* fix: allow for policyRules to be null

* test: add test case for wrong apiVersion

* test: add test case customRuleMissingIdentifier

* test: add test case customRuleMissingIdentifier

* test: add test case customRuleMissingName

* test: add test case customRuleMissingDefaultMessageOnFailure

* wip: schema validation

* fix: remove check of json schema validity, it's out of scope

* refactor: fix goimports

Co-authored-by: Dima Brusilovsky <dimabru@gmail.com>

* feat: add NSA Hardening Rules

* fix: add documentation url

* test: add unique documentation url test

* wip

* wip

* wip

* revert go.mod change

* revert go.sum change

* revert go.sum change

* changed error message text

* removed unnecessary message

* added unique field to uniqueName

* added uniqueName uniqueness validation

* removed redundant unique json schema property

* cleaned naming

* cleaned naming

* added ID uniqueness test

* cleaned the code a bit

* improved naming

* removed unneeded space

* move file path to const

* fixed lint issues

* cleaned a bit

* cleaned a bit

* improved naming

* CR: changed  map[string][]interface{}{} to nil

* CR: uncapitzled error msg

* fixed json schema

* refactor: update validation to the new validation code

* renamed to convertYamlFileToMap

* fixed validation functions

* added TestMain

* resolved lint errors

* adjusted naming

* extracted reflection logic

* CR

* refactor: revert go.mod and go.sum

* refactor: revert go.sum
Co-authored-by: teselil <tzlil@datree.com>

style: auto-reformat defaultRules.yaml

* feat: pass cli flags as headers on requests

* build: bump to version 1.4.0

* docs: fix combinations.yaml wrong api version

* docs: fix apiversion to apiVersion

* feat: first working version, show skipped files regarding k8s validation

* test: fix failing tests

* style: fix lint

* docs: add variations.yaml file and revert service.yaml

* refactor: add a check in validateResource for isValid before adding the skip warning, this doesn't effect performance but is more correct

* feat: change message for skipped k8s validation

* feat: change line breaks in case of k8s validation warning

* test: add test case for missing schema skip message

* docs: adjust combinations.yaml

* refactor: rename example file

* docs: fix typo

* style: remove empty line

* fix: add nil check before casting pointer to value, to avoid a runtime panic error

* perf: read files in parallel

* address review comments

* fix: read files more than concurrency limit
Suggested-by: shalev007 <51760613+shalev007@users.noreply.github.com>

* refactor: drill down rulesData to JUnit output printer

* refactor: drill rulesData in test cases as well

* feat: add working implementation and update test case

* style: fix lint

* style: fix imports lint error

* refactor: rename getRuleResultIfItExists to findRuleResult

Co-authored-by: teselil <tzlil@datree.com>
Co-authored-by: Roy Hadad <roy@datree.io>

* feat: save and use default rules from datree dir

* feat: add error message when cannot pull defaultrules from server
Co-authored-by: shalev avhar <shalev@datree.io>

Co-authored-by: teselil <tzlil@datree.com>

* refactor: extract output flag validation to a single function

* refactor: replace if_else with switch statement

* refactor: a small refactor

* test: add test case for empty output option

* fix: wrong failed rule counter

* fix: total skipped with more then one occurrences

* fix: non interactive output total rules passed correct number

* fix: occurrences wrong amount on print

* fix: changed occurencesCount method
Co-authored-by: shalev avhar <shalev@datree.io>
Co-authored-by: shmuelsa <shmuel@datree.io>
Co-authored-by: Yishay Mendelsohn <myishay@gmail.com>

* feat: add skip rules

* feat: add Configuration skipped properties

* refactor: split loop - extract filesData populate to its own loop

* refactor: rename ruleWithSchema to rule

* refactor: split loop - extract rulesData initialization to its own loop

* refactor: move configuration extraction to upper loop

* refactor: slide statements

* refactor: remove redundant line

* refactor: rename newFailedRule to failedRule

* refactor: extract evaluateRule function

* refactor: add jsonSchemaValidator to evaluator context and add evaluateRule to evaluator

* refactor: extract evaluateConfiguration method

* refactor: change failedRuleData to failedRule variable name

* feat: add skip for cli format

* refactor: add EvaluationResultsSummery struct

* feat: add skip rules print

* fix: add go linter formatting

* feat: add cyan color for skip rule summery

* fix: show all skipped regardless of failed rules
Co-authored-by: shalev avhar <shalev@datree.io>
Co-authored-by: Alex <alex@datree.io>
Co-authored-by: shmuelsa <95699772+shmuelsa@users.noreply.github.com>