Skip to content

Conversation

@seay28
Copy link
Collaborator

@seay28 seay28 commented Nov 24, 2025

  • changed file name
  • changed name for routing

- changed file name
- changed name for routing
@seay28 seay28 requested review from MadMax93 and schwzr November 24, 2025 13:45
Copy link
Member

@schwzr schwzr left a comment

Choose a reason for hiding this comment

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

see inline comments

| 0 | Success (no errors, or `--no-fail` was used) |
| 1 | Failure (errors found, or fatal error occurred) |

## Validation Rules
Copy link
Member

Choose a reason for hiding this comment

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

maybe we should move the list of validation rules to another page?

| `BpmnModelLinter` | Validates BPMN model structure and elements |
| `FhirResourceLinter` | Validates FHIR resources using pluggable linters |

### Linting Phases
Copy link
Member

Choose a reason for hiding this comment

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

maybe we should also move the phases to another page?

- Respects `--no-fail` flag
- Returns appropriate status code

### Design Patterns
Copy link
Member

Choose a reason for hiding this comment

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

do we need this?

- **Service Locator Pattern**: Plugin discovery via ServiceLoader
- **Builder Pattern**: Configuration and result objects

### Thread Safety
Copy link
Member

Choose a reason for hiding this comment

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

do we need this?

- Temporary context classloader used for resource access
- Result objects are immutable

## Development
Copy link
Member

Choose a reason for hiding this comment

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

maybe we should move this to another page

- Handle exceptions appropriately
- Write unit tests for new features

## Troubleshooting
Copy link
Member

Choose a reason for hiding this comment

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

move this to another page and make sure it is clearly visible (and above developer stuff)

title: DSF Linter
icon: operate
---
# DSF Linter - Complete Documentation
Copy link
Member

Choose a reason for hiding this comment

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

do we need 2 pages for this? Maybe use one page and have 2 subpages for the different validation rules?

…tion

- new sidebar update
- Api v2 linter tool redirects to Api v1 linter tool  because we currently don't have any difference
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants