Skip to content

feat: html config transform handling for MES v12 only - #786

Closed
joaoopereira wants to merge 25 commits into
developmentfrom
development-config-transform-only-v12
Closed

joaoopereira wants to merge 25 commits into
developmentfrom
development-config-transform-only-v12

Conversation

@joaoopereira

Copy link
Copy Markdown
Member

What was done

Please explain the origin/source of the bug/feat

Dependencies

Please indicate if this PullRequest has Business/GUI/other dependencies (and attach here when existing).
Delete this section if not needed.

Validation checklist

Please ensure the following conditions are met in order for this PR to be merged:

  • Code validation (careful with side-effects)
  • Integration test run provided
  • Main WorkItem is linked
  • Target branch version is correct

Rui Jorge and others added 25 commits July 7, 2026 23:21
…roject

The Data command always passed net6.0 as the target framework when
scaffolding the Actions project, regardless of MES version. This caused
the Actions project to target net6.0 even for MES 11+ where net8.0 is
expected.

Align with BusinessCommand and TestCommand by resolving the target
framework dynamically via IDependencyVersionService.DotNetTargetFramework(),
which returns net8.0 for MES >= 11 and net6.0 otherwise.

Fixes #770
- upgraded adm-zip from 0.5.12 to 0.6.0
- updated brace-expansion from 1.1.14 to 1.1.18
- upgraded fast-uri from 3.1.2 to 3.1.5
- updated js-yaml from 4.1.1 to 4.3.1 and added funding information
@joaoopereira
joaoopereira force-pushed the development-config-transform-only-v12 branch from 180a3e4 to d431ecc Compare September 7, 2026 09:33
@github-actions

github-actions Bot commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

.NET Test Results

View full test report

@github-actions

github-actions Bot commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Code Coverage

Package Line Rate Branch Rate Health
cmf 43% 43% ➖
core 71% 53% ✔
Summary 54% (5379 / 10047) 48% (2807 / 5834) ➖

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.

6 participants