Skip to content

Scrub XML and fix visualizer#277

Merged
thomas-bc merged 2 commits intodevelfrom
pr-3961
Jul 31, 2025
Merged

Scrub XML and fix visualizer#277
thomas-bc merged 2 commits intodevelfrom
pr-3961

Conversation

@thomas-bc
Copy link
Contributor

Related Issue(s)
Has Unit Tests (y/n)
Documentation Included (y/n)

Change Description

Required changes for fixing nasa/fprime#3938 on core F´ (changes to versioning.py to allow removing XML stuff)

Fixes nasa/fprime#3939

@thomas-bc thomas-bc closed this Jul 31, 2025
@thomas-bc thomas-bc reopened this Jul 31, 2025
import argparse
import sys
from pathlib import Path
import pkg_resources

Check notice

Code scanning / CodeQL

Unused import Note

Import of 'pkg_resources' is not used.

Copilot Autofix

AI 7 months ago

To fix the problem, simply remove the unused import statement for pkg_resources on line 6 of src/fprime/util/versioning.py. This will clean up the code, remove an unnecessary dependency, and improve readability. No other changes are required, as the rest of the code does not reference pkg_resources.

Suggested changeset 1
src/fprime/util/versioning.py

Autofix patch

Autofix patch
Run the following command in your local git repository to apply this patch
cat << 'EOF' | git apply
diff --git a/src/fprime/util/versioning.py b/src/fprime/util/versioning.py
--- a/src/fprime/util/versioning.py
+++ b/src/fprime/util/versioning.py
@@ -5,3 +5,3 @@
 from pathlib import Path
-import pkg_resources
+
 
EOF
@@ -5,3 +5,3 @@
from pathlib import Path
import pkg_resources


Copilot is powered by AI and may make mistakes. Always verify output.
@thomas-bc thomas-bc merged commit 0f20049 into devel Jul 31, 2025
33 of 56 checks passed
thomas-bc added a commit to tanuj-rai/fprime-tools that referenced this pull request Aug 11, 2025
* Scrub XML and fix visualizer

* formatting
@thomas-bc thomas-bc deleted the pr-3961 branch September 2, 2025 23:20
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.

Add the visualizer to CI (and fix regressions)

2 participants