Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FP-2784: define trace id type #2917

Merged
merged 3 commits into from
Oct 7, 2024
Merged

Conversation

Shivaly-Reddy
Copy link
Collaborator

@Shivaly-Reddy Shivaly-Reddy commented Oct 2, 2024

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

Change Description

Adding a trace id type for FPP.

Rationale

There is a corresponding change being PRed for fpp/main - nasa/fpp#517

Testing/Review Recommendations

Been used and tested with FP-2784 (implementation of Trace features)

Future Work

Trace feature will be added.

@Shivaly-Reddy Shivaly-Reddy requested a review from bocchino October 2, 2024 17:48
Copy link
Collaborator

@bocchino bocchino left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@Shivaly-Reddy
Copy link
Collaborator Author

@LeStarch : I don't think I have the permission to merge the approved pull request? This change needs to be merged in-sync with the corresponding fpp change. Can you please let me know what's the best way to do that?

@thomas-bc
Copy link
Collaborator

@Shivaly-Reddy The usual process for this it to merge nasa/fpp#517, pre-release FPP and upgrade the pre-release version in requirements.txt here in this PR

@Shivaly-Reddy
Copy link
Collaborator Author

Thanks @thomas-bc, Rob gave me the alpha version that would contain the expected changes. Once its released, I'll go ahead and update the requirements.txt.

@bocchino
Copy link
Collaborator

bocchino commented Oct 3, 2024

FPP v2.2.0a5 is now available.

@LeStarch LeStarch merged commit 65b9e4d into nasa:devel Oct 7, 2024
47 of 48 checks passed
@Shivaly-Reddy Shivaly-Reddy deleted the FP-2784_fpp_changes branch October 15, 2024 19:18
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.

4 participants