Skip to content

NC | lifecycle | add noncurrent days rule #8992

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

NC | lifecycle | add noncurrent days rule #8992

wants to merge 1 commit into from
+130 −8

Conversation

nadavMiz
Copy link
Contributor

@nadavMiz nadavMiz commented Apr 27, 2025 •
edited
Loading

Describe the Problem

implement POSIX noncurrent days rule

Explain the Changes

  1. add new attribute to object info for number of noncurrent days - nc_noncurrent_time
  2. implement the filter_noncurrent_days function
  3. add integration tests

Issues: Fixed #xxx / Gap #xxx

Testing Instructions:

  1. run sudo npx jest test_nc_lifecycle_posix_integration
  • Doc added/updated
  • Tests added

@nadavMiz nadavMiz requested review from romayalon and shirady April 27, 2025 15:43
@nadavMiz nadavMiz force-pushed the lifecycle_noncurrent_days branch from 90eb540 to f1b888b Compare April 27, 2025 15:45
@shirady shirady mentioned this pull request Apr 28, 2025
Open
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@romayalon romayalon Awaiting requested review from romayalon

@shirady shirady Awaiting requested review from shirady

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
size/L
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@nadavMiz