Skip to content

Fix/ignore run configurations#1460

Merged
sven1103 merged 3 commits into
developmentfrom
fix/ignore-run-configurations
Jun 19, 2026
Merged

Fix/ignore run configurations#1460
sven1103 merged 3 commits into
developmentfrom
fix/ignore-run-configurations

Conversation

@sven1103

Copy link
Copy Markdown
Contributor

Description

Issue and Traceability

Linked Task Issue

  • Closes: #

Requirement IDs Addressed

  • Requirement IDs:

Requirement Update Status

  • Requirements Updated: The corresponding functional/non-functional requirements in docs/requirements.md have been updated to reflect this implementation.
  • Justification Provided: This PR does not change externally observable behavior, so no requirement update is needed. (Provide brief justification below)

Justification (if applicable)

Changes Summary

Modified Areas

  • Domain layer (domain/model/, domain events, aggregates)
  • Application layer (application/ services, use cases)
  • Infrastructure layer (JPA repositories, external integrations, config)
  • UI / Views (views/ components, Vaadin routes)
  • Database schema (sql/, DDL changes)
  • Tests (unit, integration, or Spock specs)
  • Documentation, CI/CD, or build configuration
  • Other (describe):

Behavioral Changes

  • No behavioral changes (internal refactoring, optimization, test addition, etc.)
  • Yes, behavior changes:

Sensitive Changes Requiring Review

  • Database schema changes (sql/complete-schema.sql, table/column DDL)
  • Spring Security configuration (security/, ACL setup, authentication/authorization)
  • FAIR / RO-Crate export format (docs/fair/, RO-Crate builder logic)
  • Artemis messaging topics (JMS topic names, consumer configuration)
  • Requirement file edits (docs/requirements.md)
  • None of the above

Pre-Submission Checklist

  • Issue Linked: PR references a Task issue (see "Linked Task Issue" section above)
  • Requirements Listed: All functional (R) and non-functional (NFR) requirement IDs are listed above
  • No Constraint IDs: No C (constraint) IDs are listed in the Requirement IDs section
  • Behavior vs. Requirements: Requirement update confirmed OR explicit justification provided
  • Code Style: Code follows Google Java Style Guide (run formatter before commit)
  • Tests: New tests added or existing tests updated to cover changes
  • No Secrets: No hardcoded credentials, API keys, passwords, or secrets committed
  • Documentation: Updated relevant docs if the change affects user-facing behavior or public APIs

Notes for Reviewers

- Add GOLD color to TagColor enum (#FFF3CC fill, #A68500 text per Figma)
- Add .tag.gold CSS rule in span.css
- Fix ProjectSummaryComponent.buildTags() to include IP measurements
- Update ProjectCollectionComponent to use GOLD instead of TEAL
- Update TagFactory to use goldTag() for immunopeptidomics
@sven1103 sven1103 requested a review from a team as a code owner June 19, 2026 12:19
@github-actions github-actions Bot added the fix label Jun 19, 2026
@sven1103 sven1103 merged commit af3c0cd into development Jun 19, 2026
6 of 8 checks passed
@sven1103 sven1103 deleted the fix/ignore-run-configurations branch June 19, 2026 12:19
@sonarqubecloud

Copy link
Copy Markdown

@qbicsoftware qbicsoftware deleted a comment from gitguardian Bot Jun 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants