Lint error on empty field description

[tomyki]

Hello,
When I'm linting my data contract with this chunk of code I'm getting an error in linting

models:
  ABCD:
    description: Data ABCD view
    type: view
    fields:
      ID:
        type: number
        required: true

Error message:
Data contract is invalid, found the following errors:

1. Linter 'Objects have descriptions': Field 'ID' in model
   'ABCD' has empty description.

It is weird because description in field is said to be optional but it is throwing an error even though it is just warning,

[jon-ruckwood]

I’ve recently encountered the same issue, where only a passed result leads to a successful outcome. It seems warnings are treated the same as errors, as outlined here:

datacontract-cli/datacontract/cli.py

Lines 446 to 451 in bff771a

	if run.result == "passed":
	console.print(
	f"🟢 data contract is valid. Run {len(run.checks)} checks. Took {(run.timestampEnd - run.timestampStart).total_seconds()} seconds."
	)
	else:
	console.print("🔴 data contract is invalid, found the following errors:")

It would be helpful if the CLI handled the "passed with warnings" case more appropriately.

The lint functionality is generating a lot of false positives for us, which adds unnecessary noise.

[jochenchrist]

Could you review, if this PR would resolve your issue: #581

[jon-ruckwood]

Could you review, if this PR would resolve your issue: #581

Yes, I've tried it and it does solve my issue.

I was thinking about contributing myself, but you guys are quick – thank you!