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

style: use overwrite and not 'o' flag in Python scripts #4175

Closed
wants to merge 2 commits into from

Conversation

neteler
Copy link
Member

@neteler neteler commented Aug 13, 2024

Fixes messages like

WARNING: Please update the usage of <v.db.connect>: flag <o> has been
         renamed to <--overwrite>

in modules.
Also updates several testsuite files, accordingly.

Note that in some places overwrite=... was already present (duplication detected by pre-commit).

Fixes messages like

```bash
WARNING: Please update the usage of <v.db.connect>: flag <o> has been
         renamed to <--overwrite>
```

in modules and testsuite files.
@neteler neteler added the Python Related code is in Python label Aug 13, 2024
@neteler neteler added this to the 8.5.0 milestone Aug 13, 2024
@neteler neteler self-assigned this Aug 13, 2024
@neteler neteler changed the title python: use overwrite and not 'o' flag style: use overwrite and not 'o' flag in Python scripts Aug 13, 2024
@neteler
Copy link
Member Author

neteler commented Aug 13, 2024

Ok, this went completely wrong. Closing.

@neteler neteler closed this Aug 13, 2024
@neteler neteler removed this from the 8.5.0 milestone Aug 13, 2024
@neteler neteler added invalid This doesn't seem to be a bug and removed Python Related code is in Python labels Aug 13, 2024
@neteler neteler removed their assignment Aug 13, 2024
neteler added a commit to neteler/grass that referenced this pull request Aug 13, 2024
(replaces OSGeo#4175)

Fixes message

```bash
WARNING: Please update the usage of <v.db.connect>: flag <o> has been
         renamed to <--overwrite>
```
neteler added a commit that referenced this pull request Aug 14, 2024
(replaces #4175)

Fixes message

```bash
WARNING: Please update the usage of <v.db.connect>: flag <o> has been
         renamed to <--overwrite>
```
landam pushed a commit to landam/grass that referenced this pull request Aug 22, 2024
(replaces OSGeo#4175)

Fixes message

```bash
WARNING: Please update the usage of <v.db.connect>: flag <o> has been
         renamed to <--overwrite>
```
Mahesh1998 pushed a commit to Mahesh1998/grass that referenced this pull request Sep 19, 2024
(replaces OSGeo#4175)

Fixes message

```bash
WARNING: Please update the usage of <v.db.connect>: flag <o> has been
         renamed to <--overwrite>
```
@neteler neteler deleted the python_use_overwrite_not_o_flag branch November 25, 2024 21:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
invalid This doesn't seem to be a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant