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

Fix #44, CS_RecomputeAppChildTask wrong function key #45

Closed
wants to merge 1 commit into from

Conversation

irowebbn
Copy link

No description provided.

@chillfig
Copy link
Contributor

Hi @irowebbn , would you please add comments to this pull request? (example: #43)

@chillfig chillfig self-requested a review September 29, 2022 16:07
Copy link
Contributor

@chillfig chillfig left a comment

Choose a reason for hiding this comment

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

The assert errors produced from 'CS_RecomputeAppChildTask_Test_DefEntryId()` shown in #44 do not align with the unit test coverage results ran on main branch of nasa/cs. However, the change suggested is appropriate and maintains 100% unit coverage in nasa/cs. This pull requests requires pull request description and a contributor license agreement from the contributor before merging.

@irowebbn
Copy link
Author

irowebbn commented Oct 4, 2022

@chillfig Thanks for the help. I am waiting to hear back from my employer about permission for the CLA since this was discovered as part of my official duties.

@dzbaker
Copy link
Contributor

dzbaker commented Oct 13, 2022

@irowebbn CCB 13 October 2022: Pending CLA and PR description/linked issue.

@dzbaker
Copy link
Contributor

dzbaker commented Jan 26, 2023

26 January 2023: Ignoring while waiting on CLA.

@thnkslprpt
Copy link
Contributor

thnkslprpt commented Mar 4, 2023

The assert errors produced from 'CS_RecomputeAppChildTask_Test_DefEntryId()` shown in #44 do not align with the unit test coverage results ran on main branch of nasa/cs...

Tests failing on nasa/CS main branch as well now (see https://github.com/thnkslprpt/CS/actions/runs/4333025360/jobs/7565890664#step:11:3742).

I will submit a PR (given that this one has been stuck for 5 months) with the change identified by @irowebbn fixing the tests so that it doesn't start holding up other PRs.

@irowebbn
Copy link
Author

Superceded by #65

@irowebbn irowebbn closed this Oct 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants