Skip to content

add update-catalog-active to CLI#109

Merged
monodera merged 1 commit into
mainfrom
tickets/OBSPROC-120
Apr 5, 2025
Merged

add update-catalog-active to CLI#109
monodera merged 1 commit into
mainfrom
tickets/OBSPROC-120

Conversation

@monodera
Copy link
Copy Markdown
Contributor

@monodera monodera commented Apr 5, 2025

No description provided.

github-advanced-security[bot]
github-advanced-security AI found potential problems Apr 5, 2025
View reviewed changes
Comment thread src/targetdb/cli/cli_main.py Outdated

Check warning

Code scanning / Pylintpython3 (reported by Codacy)

Missing function or method docstring

Missing function or method docstring
Comment thread src/targetdb/utils.py
if verbose:
logger.info(
f"Updating input_catalog_id {input_catalog_id} to active={active_flag}"
)

Check warning

Code scanning / Pylintpython3 (reported by Codacy)

Variable name "df" doesn't conform to snake_case naming style

Variable name "df" doesn't conform to snake_case naming style
github-advanced-security[bot]
github-advanced-security AI found potential problems Apr 5, 2025
View reviewed changes
Comment thread src/targetdb/cli/cli_main.py Outdated

Check warning

Code scanning / Pylint (reported by Codacy)

Missing function docstring

Missing function docstring
Comment thread src/targetdb/cli/cli_main.py Outdated

Check warning

Code scanning / Pylint (reported by Codacy)

Wrong hanging indentation before block (add 4 spaces).

Wrong hanging indentation before block (add 4 spaces).
Comment thread src/targetdb/cli/cli_main.py Outdated

Check warning

Code scanning / Pylint (reported by Codacy)

Wrong hanging indentation before block (add 4 spaces).

Wrong hanging indentation before block (add 4 spaces).
Comment thread src/targetdb/cli/cli_main.py Outdated

Check warning

Code scanning / Pylint (reported by Codacy)

Wrong hanging indentation before block (add 4 spaces).

Wrong hanging indentation before block (add 4 spaces).
Comment thread src/targetdb/cli/cli_main.py Outdated

Check warning

Code scanning / Pylint (reported by Codacy)

Wrong hanging indentation before block (add 4 spaces).

Wrong hanging indentation before block (add 4 spaces).
github-advanced-security[bot]
github-advanced-security AI found potential problems Apr 5, 2025
View reviewed changes
Comment thread src/targetdb/cli/cli_main.py Outdated

Check warning

Code scanning / Prospector (reported by Codacy)

syntax error in forward annotation 'Input catalog ID to be updated.' (F722)

syntax error in forward annotation 'Input catalog ID to be updated.' (F722)
Comment thread src/targetdb/cli/cli_main.py Outdated

Check warning

Code scanning / Prospector (reported by Codacy)

syntax error in forward annotation 'Active flag to be set.' (F722)

syntax error in forward annotation 'Active flag to be set.' (F722)
Comment thread src/targetdb/cli/cli_main.py Outdated

Check warning

Code scanning / Prospector (reported by Codacy)

undefined name 'commit' (F821)

undefined name 'commit' (F821)
Comment thread src/targetdb/cli/cli_main.py Outdated

Check warning

Code scanning / Prospector (reported by Codacy)

syntax error in forward annotation 'Commit changes to the database.' (F722)

syntax error in forward annotation 'Commit changes to the database.' (F722)
Comment thread src/targetdb/cli/cli_main.py Outdated

Check warning

Code scanning / Prospector (reported by Codacy)

undefined name 'v' (F821)

undefined name 'v' (F821)
github-advanced-security[bot]
github-advanced-security AI found potential problems Apr 5, 2025
View reviewed changes
Comment thread docs/reference/cli.md
* `-t, --table [filter_name|fluxstd|input_catalog|proposal|proposal_category|sky|target|target_type]`: Table name to insert rows. [required]
* `-t, --table [filter_name|fluxstd|input_catalog|partner|pfs_arm|proposal|proposal_category|sky|target|target_type|user_pointing]`: Table name to insert rows. [required]
* `--commit`: Commit changes to the database.
* `--fetch`: Fetch data from database a the end.

Check notice

Code scanning / Remark-lint (reported by Codacy)

Warn when shortcut reference links are used.

[no-shortcut-reference-link] Use the trailing `[]` on reference links
Comment thread docs/reference/cli.md
* `-t, --table [filter_name|fluxstd|input_catalog|proposal|proposal_category|sky|target|target_type]`: Table name to insert rows. [required]
* `-t, --table [filter_name|fluxstd|input_catalog|partner|pfs_arm|proposal|proposal_category|sky|target|target_type|user_pointing]`: Table name to insert rows. [required]
* `--commit`: Commit changes to the database.
* `--fetch`: Fetch data from database a the end.

Check notice

Code scanning / Remark-lint (reported by Codacy)

Warn when references to undefined definitions are found.

[no-undefined-references] Found reference to undefined definition
Comment thread docs/reference/cli.md
* `--force`: Force download.
* `--force / --no-force`: Force download. [default: no-force]
* `--help`: Show this message and exit.

Check notice

Code scanning / Remark-lint (reported by Codacy)

Warn when shortcut reference links are used.

[no-shortcut-reference-link] Use the trailing `[]` on reference links
Comment thread docs/reference/cli.md
* `--force`: Force download.
* `--force / --no-force`: Force download. [default: no-force]
* `--help`: Show this message and exit.

Check notice

Code scanning / Remark-lint (reported by Codacy)

Warn when references to undefined definitions are found.

[no-undefined-references] Found reference to undefined definition
Comment thread docs/reference/cli.md
* `-t, --table [filter_name|fluxstd|input_catalog|proposal|proposal_category|sky|target|target_type]`: Table name to update rows. [required]
* `-t, --table [filter_name|fluxstd|input_catalog|partner|pfs_arm|proposal|proposal_category|sky|target|target_type|user_pointing]`: Table name to update rows. [required]
* `--commit`: Commit changes to the database.
* `--fetch`: Fetch data from database a the end.

Check notice

Code scanning / Remark-lint (reported by Codacy)

Warn when references to undefined definitions are found.

[no-undefined-references] Found reference to undefined definition
Comment thread docs/reference/cli.md

* `INPUT_CATALOG_ID`: Input catalog ID to be updated. [required]
* `ACTIVE_FLAG`: Active flag to be set. [required]

Check notice

Code scanning / Remark-lint (reported by Codacy)

Warn when references to undefined definitions are found.

[no-undefined-references] Found reference to undefined definition
Comment thread docs/reference/cli.md

* `INPUT_CATALOG_ID`: Input catalog ID to be updated. [required]
* `ACTIVE_FLAG`: Active flag to be set. [required]

Check notice

Code scanning / Remark-lint (reported by Codacy)

Warn when shortcut reference links are used.

[no-shortcut-reference-link] Use the trailing `[]` on reference links
Comment thread docs/reference/cli.md
* `INPUT_CATALOG_ID`: Input catalog ID to be updated. [required]
* `ACTIVE_FLAG`: Active flag to be set. [required]

**Options**:

Check notice

Code scanning / Remark-lint (reported by Codacy)

Warn when references to undefined definitions are found.

[no-undefined-references] Found reference to undefined definition
Comment thread docs/reference/cli.md
* `INPUT_CATALOG_ID`: Input catalog ID to be updated. [required]
* `ACTIVE_FLAG`: Active flag to be set. [required]

**Options**:

Check notice

Code scanning / Remark-lint (reported by Codacy)

Warn when shortcut reference links are used.

[no-shortcut-reference-link] Use the trailing `[]` on reference links
Comment thread docs/reference/cli.md

* `-c, --config TEXT`: Database configuration file in the TOML format. [required]
* `--commit`: Commit changes to the database.
* `-v, --verbose`: Verbose output.

Check notice

Code scanning / Remark-lint (reported by Codacy)

Warn when references to undefined definitions are found.

[no-undefined-references] Found reference to undefined definition
@monodera monodera merged commit 35c1568 into main Apr 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@monodera @github-advanced-security