Skip to content

Add parameter docs to README and CI/CD workflow#8

Merged
ypriverol merged 1 commit intodevfrom
feature/ci-readme-cleanup
Apr 14, 2026
Merged

Add parameter docs to README and CI/CD workflow#8
ypriverol merged 1 commit intodevfrom
feature/ci-readme-cleanup

Conversation

@ypriverol
Copy link
Copy Markdown
Member

Summary

  • Modernized README.md with full parameter reference tables, quick start examples, and build instructions
  • Created GitHub Actions CI/CD workflow (included below — requires workflow token scope to push)

README Changes

  • Full parameter reference tables covering all 30+ command-line flags, organized by category:
    • Core search (tolerance, isotope error, TDA)
    • Fragmentation and instrument settings
    • Enzyme and digestion options
    • Peptide filtering (length, charge, MS level)
    • Modifications and protocol
    • Output and performance tuning
    • Advanced parameters
  • Quick start examples for basic and TMT searches
  • Modification file format documentation with examples
  • Build-from-source instructions
  • Updated requirements to Java 17+

CI/CD Workflow

The GitHub token lacks workflow scope, so the CI file needs to be added manually. Create .github/workflows/ci.yml:

name: CI

on:
  push:
    branches: [ master, dev ]
  pull_request:
    branches: [ master, dev ]

jobs:
  build:
    runs-on: ubuntu-latest
    strategy:
      matrix:
        java: [ '17', '21' ]

    steps:
      - uses: actions/checkout@v4

      - name: Set up JDK ${{ matrix.java }}
        uses: actions/setup-java@v4
        with:
          java-version: ${{ matrix.java }}
          distribution: 'temurin'
          cache: 'maven'

      - name: Build with Maven
        run: mvn compile -q

      - name: Run tests
        run: mvn test

      - name: Package JAR
        run: mvn package -DskipTests -q

      - name: Upload JAR artifact
        if: matrix.java == '17'
        uses: actions/upload-artifact@v4
        with:
          name: MSGFPlus-jar
          path: target/MSGFPlus.jar
          retention-days: 30

Test plan

  • mvn compile — clean compilation
  • mvn test — 137 tests run, 0 failures, 0 errors
  • README renders correctly with parameter tables

🤖 Generated with Claude Code

Rewrite README.md:
- Full parameter reference tables covering all 30+ flags organized
  by category (core search, fragmentation, enzyme, filtering, etc.)
- Quick start examples for basic and TMT searches
- Modification file format documentation with examples
- Build-from-source instructions
- Updated requirements to Java 17+

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 13, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 4dd7c6db-bfb9-422f-b94e-2e5ce1c8e996

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/ci-readme-cleanup

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@ypriverol ypriverol merged commit 3909fe8 into dev Apr 14, 2026
1 check passed
@ypriverol ypriverol deleted the feature/ci-readme-cleanup branch April 16, 2026 16:36
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.

1 participant