graphIO: serialize username that saved the file#3134
Conversation
There was a problem hiding this comment.
Code Review
This pull request adds the current username to the serialized file header using getpass.getuser(). Feedback suggests wrapping the getpass.getuser() call in a try-except block to prevent potential crashes in environments where user information is unavailable, such as containerized or headless systems.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
Codecov Report❌ Patch coverage is
❌ Your patch check has failed because the patch coverage (66.66%) is below the target coverage (70.00%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## develop #3134 +/- ##
===========================================
- Coverage 85.33% 85.32% -0.01%
===========================================
Files 73 73
Lines 11414 11420 +6
===========================================
+ Hits 9740 9744 +4
- Misses 1674 1676 +2 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Description
In the serialized file, save the username that last saved the file.
It could be interesting for stat purposes, we can get the modification date with
statbut not the username easily.