Skip to content

feature: added attributes support for definition generator and RQ8 - #147

Merged
mortenzwarenstein merged 15 commits into
masterfrom
validate-attribute-table-definitions
Sep 26, 2025
Merged

mortenzwarenstein merged 15 commits into
masterfrom
validate-attribute-table-definitions

Conversation

@mortenzwarenstein

Copy link
Copy Markdown
Collaborator

Description

What have you changed and why?

Type of change

(Remove irrelevant options)

  • Minor change (typo, formatting, version bump)
  • New feature
  • Improvement of existing feature
  • Bugfix
  • Refactoring
  • Configuration change
  • Breaking change

Checklist:

  • I've double-checked the code in this PR myself
  • I've left the code better than before (boy scout rule)
  • The code is readable, comments are added that explain hard or non-obvious parts.
  • I've expanded/improved the (unit) tests, when applicable
  • I've run (unit) tests that prove my solution works
  • There's no sensitive information like credentials in my PR

Comment thread geopackage_validator/validate.py Outdated
Comment thread geopackage_validator/validations/attribute_table_check.py Outdated
Comment thread tests/validations/test_attribute_table_check.py Outdated
@mortenzwarenstein
mortenzwarenstein marked this pull request as ready for review September 26, 2025 07:19
@mortenzwarenstein
mortenzwarenstein merged commit 0bbb5df into master Sep 26, 2025
7 checks passed
@rkettelerij
rkettelerij deleted the validate-attribute-table-definitions branch September 26, 2025 07:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants