[Fixes #14344] Pack rdf files in build distribution#14345
Conversation
There was a problem hiding this comment.
Code Review
This pull request attempts to include RDF thesauri files by adding a [tool.setuptools.package-data] configuration to pyproject.toml. However, this configuration will be ignored because setup.py overrides it. It is recommended to remove this block and instead update MANIFEST.in to include the .rdf files.
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.
There was a problem hiding this comment.
Pull request overview
Updates GeoNode’s setuptools packaging configuration so RDF thesaurus files stored under **/thesauri/*.rdf are included in built distributions, enabling the thesaurus autoload feature to work when GeoNode is installed from a packaged artifact.
Changes:
- Adds a
tool.setuptools.package-dataentry to include**/thesauri/*.rdfunder thegeonodepackage.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #14345 +/- ##
===========================================
+ Coverage 34.71% 74.95% +40.23%
===========================================
Files 982 983 +1
Lines 60496 60615 +119
Branches 8247 8266 +19
===========================================
+ Hits 21002 45434 +24432
+ Misses 38332 13351 -24981
- Partials 1162 1830 +668 🚀 New features to boost your workflow:
|
Fixes #14344
Checklist
For all pull requests:
The following are required only for core and extension modules (they are welcomed, but not required, for contrib modules):
Submitting the PR does not require you to check all items, but by the time it gets merged, they should be either satisfied or inapplicable.