Skip to content

fix(urls): open SAM-BIM/SAM from the System Results source-code action - #38

Merged
michaldengusiak merged 2 commits into
sow/2026-Q4from
fix/sam-bim-runtime-urls-q4
Oct 6, 2026
Merged

michaldengusiak merged 2 commits into
sow/2026-Q4from
fix/sam-bim-runtime-urls-q4

Conversation

@michaldengusiak

@michaldengusiak michaldengusiak commented Oct 6, 2026 •

Copy link
Copy Markdown

Summary

The System Results component's "source code" menu action (OnSourceCodeClick) opened https://github.com/HoareLea/SAM; it now opens https://github.com/SAM-BIM/SAM, like the shared SAM Core components (SAMAnalyticalSystemResults.cs, one line). Source-only: no workflow, .gitmodules, gitlink, master, sow/2026-Q3 or icon-redesign change. Kernel/AssemblyInfo.cs author/contact strings are provenance and are unchanged (owner decision).

PR record: docs/SAM-BIM-RuntimeUrls-Q4.md (final commit on this branch) holds the full record; this description summarises it.

  • Status: open, not merged. Files changed: SAMAnalyticalSystemResults.cs (1 line). Unresolved: none introduced.
  • Next step: maintainer review and merge into sow/2026-Q4 once CI is green; then the PROJECT_PROGRESS.md closeout as a direct docs commit on sow/2026-Q4.

Validation

  • git diff --check clean; diff reviewed (1 insertion, 1 deletion).
  • msbuild SAM_Systems.sln -p:Configuration=Release with APPDATA/USERPROFILE redirected: 0 errors; SAM.Analytical.Grasshopper.Systems.gha contains https://github.com/SAM-BIM/SAM and no https://github.com/HoareLea/SAM.
  • SAM.Analytical.Systems.Tests were not run locally ; the change touches no code path they cover. CI (build, test, spdx) runs them on this PR.

Not merged by this change; merge is the maintainer's call.

🤖 Generated with Claude Code

OnSourceCodeClick opened HoareLea/SAM; it now opens SAM-BIM/SAM, like the shared
SAM Core components.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

Record document required by AGENTS.md for this PR: old HoareLea and new
SAM-BIM destinations, why, decisions, validation, Q4 base and PR number.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@michaldengusiak
michaldengusiak merged commit 0bda639 into sow/2026-Q4 Oct 6, 2026
2 checks passed
@michaldengusiak
michaldengusiak deleted the fix/sam-bim-runtime-urls-q4 branch October 6, 2026 21:54
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