Skip to content
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

v.external: delete duplicated layer requirement definition #3902

Conversation

pesekon2
Copy link
Contributor

options->layer->required = NO; is contained just a few rows above

@pesekon2 pesekon2 self-assigned this Jun 19, 2024
@pesekon2 pesekon2 requested a review from landam June 19, 2024 14:02
@pesekon2 pesekon2 added vector Related to vector data processing C Related code is in C labels Jun 19, 2024
Copy link
Member

@echoix echoix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a way to do a search automatically in all the code base for issues like this?
Good catch!

@echoix echoix merged commit 1efff48 into OSGeo:main Jun 21, 2024
24 checks passed
cyliang368 pushed a commit to cyliang368/grass that referenced this pull request Jun 22, 2024
@pesekon2 pesekon2 deleted the v_external_delete_duplicated_layer_requirement_option branch June 22, 2024 05:38
@neteler neteler added this to the 8.5.0 milestone Jun 25, 2024
@a0x8o a0x8o mentioned this pull request Jun 27, 2024
kritibirda26 pushed a commit to kritibirda26/grass that referenced this pull request Jun 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C Related code is in C module vector Related to vector data processing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants