-
Notifications
You must be signed in to change notification settings - Fork 16
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 layer naming in LayerInfos #613
Conversation
Reviewer's Guide by SourceryThis pull request fixes the layer naming validation logic in the _validate_layers method. Instead of using the f.is_named() method to check if a layer is named, the code now directly checks the f.name attribute, ensuring that unnamed layers are correctly assigned a name based on their field identifier. Flow diagram for updated layer naming validationflowchart TD
A[Start _validate_layers method]
B{Is f.name set?}
C[Set f.name = field_name]
D[Check if f.layer == -1 or f.datatype == -1]
E[Raise InvalidLayerError]
F[Layer validation successful]
A --> B
B -- No --> C
B -- Yes --> D
C --> D
D -- Error Condition Met --> E
D -- Otherwise --> F
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We've reviewed this pull request using the Sourcery rules engine. If you would also like our AI-powered code review then let us know.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #613 +/- ##
==========================================
- Coverage 65.91% 65.89% -0.02%
==========================================
Files 64 64
Lines 9695 9695
Branches 1805 1805
==========================================
- Hits 6390 6389 -1
Misses 2780 2780
- Partials 525 526 +1 ☔ View full report in Codecov by Sentry. |
Summary by Sourcery
Bug Fixes: