Fix cluster ID collision: rename Oracle Forms cluster from C12 to C13#39
Open
devin-ai-integration[bot] wants to merge 2 commits intomainfrom
Open
Fix cluster ID collision: rename Oracle Forms cluster from C12 to C13#39devin-ai-integration[bot] wants to merge 2 commits intomainfrom
devin-ai-integration[bot] wants to merge 2 commits intomainfrom
Conversation
C12 was already assigned to .NET/Angular Containerized Decomposition. Also adds C13 cluster definition and per-repo entries to upstream-map.yaml.
Contributor
Author
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
5 tasks
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
C12 (.NET/Angular Containerized Decomposition) was already defined in upstream-map.yaml and referenced by 4 repo entries but was absent from the summary table in repos.md.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Fixes a cluster ID collision introduced in #20. The Oracle Forms HRMS Modernization cluster was incorrectly assigned C12, which was already used by the .NET/Angular Containerized Decomposition cluster (4 repos in
repos.md:361-394, defined inupstream-map.yaml:178).Changes:
catalog/repos.md: Renamed C12 → C13 in the cluster table row and both repo entries (ts-plsql-oracle-forms-legacy-codebase,uc-legacy-modernization-oracle-forms-to-java)catalog/upstream-map.yaml: Added C13 cluster definition and two per-repo entries (inserted in alphabetical order)Updates since last revision
repos.md. This row was absent before Add Oracle Forms modernization workshop (modules, event, unified track) #20 as well — the table now has a complete C1–C13 sequence with no gaps.Review & Testing Checklist for Human
C12references to Oracle Forms — all 3 occurrences inrepos.mdwere updated, but verify no module or event files reference the old cluster IDrepos.md) are complete and sequential from C1 through C13upstream-map.yamlare in correct alphabetical order relative to their neighbors (ts-plsql-*betweents-java-*andts-sas-*;uc-legacy-modernization-oracle-*afteruc-legacy-modernization-cobol-*)Notes
upstream-map.yamlare untouched — only the summary table row was added.Link to Devin session: https://partner-workshops.devinenterprise.com/sessions/6abcc66c28ee405c86876cb33da2f3a9
Requested by: @bsmitches