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

Add code, CLI for interacting with TDC CKAN #38

Open
wants to merge 18 commits into
base: main
Choose a base branch
from

Conversation

khaeru
Copy link
Contributor

@khaeru khaeru commented Dec 13, 2024

Now that the TDC CKAN instances (production and development) are running, this adds PR code and command-line tools for interacting with these via the Client class created in #35.

Also:

  • Add a test mark to prevent running tests against either CKAN instance during GitHub Actions CI workflows.
  • Add a CLI group for testing utilities, tdc test.
    • Add a command tdc test wipe to remove all CKAN packages from the "TDCI Test Organization" used by the test suite.

How to review

Try some or all of the following:

  1. In a terminal, run: tdc ckan tag_list.
  2. Show the help for a given command: tdc ckan help_show name=tag_list.
  3. Run a command with some arguments: tdc ckan tag_list all_fields=true.
  4. Run against the development instance: tdc ckan --instance=dev tag_list all_fields=true.
  5. Add an API token to your keyring, then make a query that requires authentication:
    keyring set transport-data api-token-prod
    keyring set transport-data api-token-dev
    tdc ckan user_list
    

Also:

  • Read the added documentation.
  • Run the test suite.

PR checklist

  • Checks all ✅
  • Update documentation
  • Update doc/whatsnew.rst

@khaeru khaeru self-assigned this Dec 13, 2024
@khaeru khaeru added the enh New feature or request label Dec 13, 2024
@khaeru khaeru force-pushed the enh/ckan-sdmx-crosswalk branch from 51c3a93 to 178f1ba Compare December 13, 2024 18:19
@khaeru khaeru force-pushed the enh/ckan-sdmx-crosswalk branch from 178f1ba to 4df5f7f Compare December 17, 2024 09:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enh New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant