Skip to content

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

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
michaldengusiak merged 2 commits into from
Oct 6, 2026
Merged

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

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
b42d7c3
fix(urls): open SAM-BIM/SAM from the System Results "source code" action
michaldengusiak Oct 6, 2026
ddce746
docs: add the SAM-BIM runtime URLs PR record
michaldengusiak Oct 6, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion Grasshopper/SAM.Analytical.Grasshopper.Systems/Component/SAMAnalyticalSystemResults.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -306,7 +306,7 @@ void IGH_VariableParameterComponent.VariableParameterMaintenance() { }

public virtual void OnSourceCodeClick(object sender = null, object e = null)
{
Process.Start("https://github.com/HoareLea/SAM");
Process.Start("https://github.com/SAM-BIM/SAM");
}

public string ComponentVersion
Expand Down
44 changes: 44 additions & 0 deletions docs/SAM-BIM-RuntimeUrls-Q4.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,44 @@
# SAM-BIM runtime URLs (Q4) - SAM_Systems

PR: SAM-BIM/SAM_Systems#38. Branch `fix/sam-bim-runtime-urls-q4` -> base `sow/2026-Q4` (Q4 base `edc8d0d`). Record date: 2026-10-06.

## Current status

PR open, **not merged**. Source-only, minimum change: the "source code" menu action of one Grasshopper component opened
the HoareLea repository and now opens SAM-BIM/SAM. No `.gitmodules`, gitlink, workflow, `master`, `sow/2026-Q3` or icon-redesign change.

## Work completed

| File | Old HoareLea destination | New SAM-BIM destination |
|---|---|---|
| `Grasshopper/SAM.Analytical.Grasshopper.Systems/Component/SAMAnalyticalSystemResults.cs` (`OnSourceCodeClick`) | `https://github.com/HoareLea/SAM` | `https://github.com/SAM-BIM/SAM` |

## Why the change is required

SAM-BIM is now the authoritative development ecosystem and HoareLea is no longer the synchronised operational source, so
the component's "source code" action should open the maintained repository. It matches the shared `OnSourceCodeClick` in SAM
(`SAMCoreInspect`, `SAMCoreFilterByType`) and SAM's `AppendSourceCodeAdditionalMenuItem`, which already uses `github.com/SAM-BIM`.

## Decisions and assumptions

- `Kernel/AssemblyInfo.cs` author/contact strings are provenance metadata and are unchanged (owner decision: KEEP).
- This file already carries the SPDX header, so no header change was needed.

## Files changed

`SAMAnalyticalSystemResults.cs` (1 line) and this record.

## Validation

- `git diff --check` clean; diff reviewed.
- `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. PR CI (`build`, `spdx`) green; see the PR.

## Unresolved issues, risks

- None introduced by this change.

## Exact next step

Merge into `sow/2026-Q4` after green CI (maintainer-approved). After merge, the `PROJECT_PROGRESS.md` closeout is a direct docs commit on `sow/2026-Q4` (never on this branch).
Loading