Contributing

Thank you for your interest in contributing to FSH Lint!

Ways to Contribute

• Report bugs
• Suggest features
• Improve documentation
• Write code
• Create custom rules
• Help others in discussions

Getting Started

Prerequisites

• Rust 1.80 or later
• Git
• Familiarity with FSH (FHIR Shorthand)

Development Setup

1. Fork the repository:

gh repo fork octofhir/fsh-lint-rs

2. Clone your fork:

git clone https://github.com/YOUR_USERNAME/fsh-lint-rs.git
cd fsh-lint-rs

3. Build the project:

cargo build --workspace

4. Run tests:

cargo test --workspace

Development Workflow

1. Create a Branch

git checkout -b feature/my-feature
# or
git checkout -b fix/issue-123

2. Make Changes

Follow the project structure:

• crates/fsh-lint-core - Core linting engine
• crates/fsh-lint-rules - Rule engine and built-in rules
• crates/fsh-lint-cli - Command-line interface
• crates/fsh-lint-devtools - Developer tools

3. Write Tests

Add tests for new functionality:

#[cfg(test)]
mod tests {
    use super::*;

    #[test]
    fn test_my_feature() {
        // Test implementation
    }
}

4. Run Tests

# Run all tests
cargo test --workspace

# Run specific test
cargo test --package fsh-lint-core --test my_test

# With output
cargo test -- --nocapture

5. Format Code

cargo fmt --all
cargo clippy --all-targets --all-features

6. Commit Changes

Follow conventional commits:

git add .
git commit -m "feat: add new rule for profile validation"
git commit -m "fix: correct cardinality checking logic"
git commit -m "docs: update installation guide"

Commit types:

• feat: New feature
• fix: Bug fix
• docs: Documentation
• test: Tests
• refactor: Code refactoring
• perf: Performance improvement
• chore: Maintenance

7. Push and Create PR

git push origin feature/my-feature

Then create a pull request on GitHub.

Code Guidelines

Rust Style

Follow Rust API Guidelines:

• Use descriptive names
• Prefer iterators over loops
• Use Result for error handling
• Document public APIs
• Write integration tests

Documentation

• Add doc comments to public APIs
• Include examples in doc comments
• Update user documentation
• Add changelog entries

Testing

• Write unit tests for new code
• Add integration tests for features
• Include golden file tests for parser
• Test error cases

Adding a New Rule

1. Create rule file in crates/fsh-lint-rules/src/builtin/:

use fsh_lint_core::{LintContext, Diagnostic, Severity};
use crate::Rule;

pub struct MyRule;

impl Rule for MyRule {
    fn name(&self) -> &str {
        "category/my-rule"
    }

    fn category(&self) -> RuleCategory {
        RuleCategory::Style
    }

    fn severity(&self) -> Severity {
        Severity::Warning
    }

    fn lint(&self, context: &mut LintContext) {
        // Implementation
    }
}

2. Register in crates/fsh-lint-rules/src/lib.rs

3. Add tests in crates/fsh-lint-rules/tests/

4. Add documentation in docs/

Review Process

1. Automated checks run (tests, clippy, fmt)
2. Maintainer reviews code
3. Address feedback
4. Approval and merge

Community

• GitHub Discussions: Ask questions
• GitHub Issues: Report bugs
• Pull Requests: Contribute code

License

By contributing, you agree your code will be licensed under MIT or Apache-2.0.

Questions?

Open a discussion on GitHub or reach out to maintainers.