Skip to content

Commit a906a95

Browse files
bedaHovorkaclaude
andcommitted
fix(review): address Copilot review feedback on PR #357
- Replace placeholder references "Issue #XXX, PR #358" with correct "Issue #80, PR #357" in XMLContextFactory.kt (both saveContext KDocs) and XMLContextFactoryOutputStreamTest.kt (@DisplayName annotation) - Delete docs/issues/issue-80.md stub (redirect to issue_80.md); README.md already links to the canonical issue_80.md file Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent f86e238 commit a906a95

3 files changed

Lines changed: 3 additions & 6 deletions

File tree

core/src/jvmMain/kotlin/cz/vutbr/fit/interlockSim/xml/XMLContextFactory.kt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -252,7 +252,7 @@ class XMLContextFactory : EditingContextFactory {
252252
/**
253253
* Saves an editing context to an output stream.
254254
*
255-
* Pre-save validation (Issue #XXX, PR #358):
255+
* Pre-save validation (Issue #80, PR #357):
256256
* - Validates InOut count before serialization
257257
* - Prevents saving invalid contexts (< MIN_INOUT_ELEMENTS InOuts)
258258
* - Returns false if validation fails
@@ -293,7 +293,7 @@ class XMLContextFactory : EditingContextFactory {
293293
/**
294294
* Saves an editing context to a file.
295295
*
296-
* Pre-save validation (Issue #XXX, PR #358):
296+
* Pre-save validation (Issue #80, PR #357):
297297
* - Validates InOut count before serialization
298298
* - Prevents saving invalid contexts (< MIN_INOUT_ELEMENTS InOuts)
299299
* - Returns false if validation fails

desktop-ui/src/test/kotlin/cz/vutbr/fit/interlockSim/xml/XMLContextFactoryOutputStreamTest.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -538,7 +538,7 @@ class XMLContextFactoryOutputStreamTest : KoinTestBase() {
538538
}
539539

540540
@Nested
541-
@DisplayName("Pre-Save Validation (Issue #XXX, PR #358)")
541+
@DisplayName("Pre-Save Validation (Issue #80, PR #357)")
542542
inner class PreSaveValidationTests {
543543
@Test
544544
@DisplayName("saveContext to File rejects context with 0 InOuts")

docs/issues/issue-80.md

Lines changed: 0 additions & 3 deletions
This file was deleted.

0 commit comments

Comments
 (0)