Skip to content

ci: replace msvc-dev-cmd with WIP vanilla bat - #409

Merged
VJanKraemer merged 1 commit into
mainfrom
remove_msvc_cmd
Aug 25, 2026
Merged

ci: replace msvc-dev-cmd with WIP vanilla bat#409
VJanKraemer merged 1 commit into
mainfrom
remove_msvc_cmd

Conversation

@VJanKraemer

Copy link
Copy Markdown
Contributor

Subject

redudece third party dependencies in CI

Description

msvc-dev-cmd is the only (non github) third party depency we have in our CI. And while not critical we want to experiment with vanilla setups for the CI

Instructions for review / testing

Developer checklist (address before review)

  • Changelog.md updated
  • Prepared update for depending repositories
  • Documentation updated (public API changes only)
  • API docstrings updated (public API changes only)
  • Rebase → commit history clean
  • Squash and merge → proper PR title

Comment thread SilKit/ci/setup_msvc_env.py Outdated
Comment thread SilKit/ci/setup_msvc_env.py Outdated
Comment thread SilKit/ci/setup_msvc_env.py Outdated
Comment thread SilKit/ci/setup_msvc_env.py
@VJanKraemer
VJanKraemer force-pushed the remove_msvc_cmd branch 3 times, most recently from a2db728 to 3107ee5 Compare August 20, 2026 13:42

@MariusBgm MariusBgm left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@VJanKraemer
VJanKraemer force-pushed the remove_msvc_cmd branch 2 times, most recently from 52aa171 to f19d056 Compare August 25, 2026 11:27
In order to minimize 3rd party dependencies we replaced the msvc-dev-cmd
action with a python script.

Signed-off-by: Jan Kraemer <jan.kraemer@vector.com>
@VJanKraemer
VJanKraemer force-pushed the remove_msvc_cmd branch 2 times, most recently from f19d056 to 678088a Compare August 25, 2026 12:01
@VJanKraemer
VJanKraemer merged commit 93416e4 into main Aug 25, 2026
23 of 28 checks passed
@VJanKraemer
VJanKraemer deleted the remove_msvc_cmd branch August 25, 2026 12:38
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.

3 participants