Skip to content

679 configure cryptolib#771

Merged
zlynch2 merged 6 commits into
devfrom
679-configure-cryptolib
Aug 8, 2025
Merged

679 configure cryptolib#771
zlynch2 merged 6 commits into
devfrom
679-configure-cryptolib

Conversation

@Donnie-Ice
Copy link
Copy Markdown
Contributor

@Donnie-Ice Donnie-Ice commented Jul 31, 2025

Summary of PR goals.

How to test?

  • modify or remove port values in scripts/env.sh
  • make sim for radio ports and make gsw for cryptolib ports
  • Has default values, so shouldn't fail if removed

Submodule PRs and actions prior to closing this:

Closes #679 .

@Donnie-Ice Donnie-Ice requested review from jlucas9 and zlynch2 July 31, 2025 17:44
@Donnie-Ice Donnie-Ice self-assigned this Jul 31, 2025
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jul 31, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.63%. Comparing base (5226cb2) to head (0133d63).
⚠️ Report is 17 commits behind head on dev.

Additional details and impacted files
@@            Coverage Diff             @@
##              dev     #771      +/-   ##
==========================================
- Coverage   73.64%   73.63%   -0.01%     
==========================================
  Files         761      761              
  Lines       93984    93985       +1     
  Branches     7063     7063              
==========================================
- Hits        69210    69206       -4     
- Misses      24774    24779       +5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@kevincbruce kevincbruce self-requested a review August 1, 2025 19:04
Copy link
Copy Markdown
Contributor

@kevincbruce kevincbruce left a comment

Choose a reason for hiding this comment

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

Changed ports around and verified working with cfdp/cryptolib tests.

@Will-Matheny Will-Matheny self-requested a review August 5, 2025 18:07
Copy link
Copy Markdown

@Will-Matheny Will-Matheny left a comment

Choose a reason for hiding this comment

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

Used older JSTAR laptop, with and without shared folders. Deleted ports, and ran make, and still connected to cryptolib.

@zlynch2 zlynch2 merged commit 9e69add into dev Aug 8, 2025
3 checks passed
@zlynch2 zlynch2 deleted the 679-configure-cryptolib branch September 29, 2025 15:10
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.

5 participants