Skip to content

chore: move itc script to ps1 file#1747

Merged
scottdover merged 7 commits intomainfrom
feat/split-script-2
Jan 15, 2026
Merged

chore: move itc script to ps1 file#1747
scottdover merged 7 commits intomainfrom
feat/split-script-2

Conversation

@scottdover
Copy link
Contributor

@scottdover scottdover commented Jan 7, 2026

Summary:
This moves our itc script into a ps1 file. To accomplish this, the following changes were made:

  • Tags & line codes were moved to a json file that our .ps1 script and .ts files could read
  • A new file, itc.ps1 was created with our powershell code from script.ts
    • Changes were made such that interop library path is now read from an environment variable,
      as is all of our tags & line codes
  • GetInteropDirectory.psm1 was created as a shareable module to load sas interop dlls

As a result:

  • We now have a color-coded, format-able powershell script instead of a string for itc connections
  • itc-connection-test.ps1 has been updated to use this new script (and the shared interop module)

Testing:

  • Make sure npm run watch generates/watches for file changes (with empty client/dist directory)
    • Make sure itc files get updated as expected
  • Make sure general itc functionality works as expected
  • Made sure itc-connection-test ran successfully with the instructions provided

Scott Dover added 6 commits January 7, 2026 09:22
Signed-off-by: Scott Dover <Scott.Dover@sas.com>
Signed-off-by: Scott Dover <Scott.Dover@sas.com>
Signed-off-by: Scott Dover <Scott.Dover@sas.com>
Signed-off-by: Scott Dover <Scott.Dover@sas.com>
Signed-off-by: Scott Dover <Scott.Dover@sas.com>
Signed-off-by: Scott Dover <Scott.Dover@sas.com>
@scottdover scottdover marked this pull request as ready for review January 7, 2026 18:07
@scottdover scottdover marked this pull request as draft January 12, 2026 14:42
Signed-off-by: Scott Dover <Scott.Dover@sas.com>
@scottdover scottdover marked this pull request as ready for review January 12, 2026 14:54
@scottdover scottdover added this to the 1.19.0 milestone Jan 15, 2026
@dahils dahils added testing Test the pull requests testing-complete Complete the pull requests testing and removed verification-needed testing Test the pull requests labels Jan 15, 2026
@dahils dahils assigned scottdover and unassigned dahils Jan 15, 2026
@scottdover scottdover merged commit 316a63d into main Jan 15, 2026
3 checks passed
@scottdover scottdover deleted the feat/split-script-2 branch January 15, 2026 18:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

testing-complete Complete the pull requests testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants