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.

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

Fix var resident to default to False #686

Merged
merged 2 commits into from
Apr 17, 2024
Merged

Fix var resident to default to False #686

merged 2 commits into from
Apr 17, 2024

Conversation

narimantos
Copy link
Contributor

@narimantos narimantos changed the title Fix var resident to default to False Draft: Fix var resident to default to False Apr 16, 2024
@narimantos narimantos marked this pull request as draft April 16, 2024 14:09
@narimantos narimantos requested a review from pyrco April 16, 2024 15:03
@narimantos narimantos self-assigned this Apr 16, 2024
@narimantos narimantos changed the title Draft: Fix var resident to default to False Fix var resident to default to False Apr 16, 2024
@narimantos narimantos marked this pull request as ready for review April 16, 2024 15:04
@cecinestpasunepipe cecinestpasunepipe requested review from cecinestpasunepipe and removed request for pyrco April 17, 2024 10:58
@narimantos narimantos merged commit 9e0608a into main Apr 17, 2024
16 checks passed
@narimantos narimantos deleted the DIS-2984-resident-False branch April 17, 2024 11:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cecinestpasunepipe cecinestpasunepipe cecinestpasunepipe approved these changes

Assignees

@narimantos narimantos

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

In MFT output the resident field is sometimes "None" but should always be a boolean
2 participants
@narimantos @cecinestpasunepipe