Skip to content

Postgis - Hide data without visibilities - #10840

Merged
elahrvivaz merged 4 commits into
locationtech:mainfrom
elahrvivaz:emilio/pg_vis_hide_empty
Sep 25, 2026
Merged

elahrvivaz merged 4 commits into
locationtech:mainfrom
elahrvivaz:emilio/pg_vis_hide_empty

Conversation

@elahrvivaz

Copy link
Copy Markdown
Contributor

No description provided.

@elahrvivaz elahrvivaz added this to the 6.0.0 milestone Sep 22, 2026
@cwdobbins

Copy link
Copy Markdown
Contributor

Changeset looks good overall- one super-teeny finding for your consideration:

The default IGNORE_PATHS="docs/" will also ignore docs/pom.xml, which the _binary-dist_ build needs to order its modules (or build.xml/requirements.txt`), so a PR that needed to update the docs 'build' components would also fail to build- Could narrow the scope to documentation content only, i.e. *.rst, to honor the intent of the change if you think it matters.

@elahrvivaz

Copy link
Copy Markdown
Contributor Author

Changeset looks good overall- one super-teeny finding for your consideration:

The default IGNORE_PATHS="docs/" will also ignore docs/pom.xml, which the _binary-dist_ build needs to order its modules (or build.xml/requirements.txt`), so a PR that needed to update the docs 'build' components would also fail to build- Could narrow the scope to documentation content only, i.e. *.rst, to honor the intent of the change if you think it matters.

Thanks, I think that's fine as the 'docs' CI will still run it - the dist modules take the output but don't care about any actual changes to the docs for their own build.

@elahrvivaz
elahrvivaz merged commit 18decb6 into locationtech:main Sep 25, 2026
94 checks passed
@elahrvivaz
elahrvivaz deleted the emilio/pg_vis_hide_empty branch September 25, 2026 11:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants