Skip to content

Remove swupdate wfx config#589

Merged
fmoessbauer merged 1 commit into
siemens:nextfrom
bniedermayr:refactor/swupdate-wfx-config
Jun 3, 2026
Merged

Remove swupdate wfx config#589
fmoessbauer merged 1 commit into
siemens:nextfrom
bniedermayr:refactor/swupdate-wfx-config

Conversation

@bniedermayr

Copy link
Copy Markdown
Contributor

Commit 468627ff of isar-cip-core introduced the wfx backend enablement by just setting the WFX_URL variable, thus the custom recipe in mtda is not required anymore.

@github-actions github-actions Bot requested a review from chombourger May 31, 2026 22:13
@bniedermayr bniedermayr changed the base branch from main to next May 31, 2026 22:14
@bniedermayr bniedermayr changed the title Refactor/swupdate wfx config Remove swupdate wfx config May 31, 2026
@bniedermayr bniedermayr force-pushed the refactor/swupdate-wfx-config branch 3 times, most recently from 361c014 to cc0d975 Compare May 31, 2026 22:39

@fmoessbauer fmoessbauer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice cleanup, thanks!

@fmoessbauer

Copy link
Copy Markdown
Member

@bniedermayr please also fix the spellcheck by adding the mentioned words to .github/wordlist.txt.

@bniedermayr bniedermayr force-pushed the refactor/swupdate-wfx-config branch from cc0d975 to 7db2674 Compare June 1, 2026 08:38
@bniedermayr

Copy link
Copy Markdown
Contributor Author

Just updated .github/wordlist.txt. Hope it works now. I can't trigger the check again (probably only maintainers).

@bniedermayr bniedermayr requested a review from fmoessbauer June 1, 2026 08:53
@fmoessbauer fmoessbauer added check and removed check labels Jun 1, 2026
@bniedermayr

bniedermayr commented Jun 1, 2026

Copy link
Copy Markdown
Contributor Author

The current CI error should be fixed with #586

@fmoessbauer fmoessbauer added check and removed check labels Jun 1, 2026
@chombourger chombourger removed the check label Jun 3, 2026
@chombourger

Copy link
Copy Markdown
Collaborator

The current CI error should be fixed with #586

can we rebase now that #586 is merged?
and thanks for this MR - so cool!

Commit 468627ff of isar-cip-core introduced the wfx backend enablement
by just setting the `WFX_URL` variable, thus the custom recipe in mtda
is not required anymore.

Signed-off-by: Benedikt Niedermayr <benedikt.niedermayr@siemens.com>
@bniedermayr bniedermayr force-pushed the refactor/swupdate-wfx-config branch from 7db2674 to 0362487 Compare June 3, 2026 11:02
@github-actions github-actions Bot requested a review from chombourger June 3, 2026 11:02
@fmoessbauer fmoessbauer merged commit 097a5b9 into siemens:next Jun 3, 2026
4 checks passed
@fmoessbauer fmoessbauer removed the check label Jun 3, 2026
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