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

Metadata File #53

Merged
merged 7 commits into from
Nov 15, 2022
Merged

Metadata File #53

merged 7 commits into from
Nov 15, 2022

Conversation

buehlere
Copy link
Collaborator

@buehlere buehlere commented Oct 11, 2022

✔️ add count to beagle_cli config file. This extra field allows the max page number to be returned if --all is specified in a list commands. This is useful if you don't want to have to tune the page parameter to return all data for a given call. It may require users to re-fresh their config file.
✔️ add cleanup.py. This script cleans up the files api metadata into a format that usable to the CCI team.
✔️ add column_names .txt to use for check format check in cleanup.py. Update README.

A few thoughts:

  • I think it's probably a good idea to maybe re-organize where cleanup.py is in the repo. But not exactly sure the best way to do this?
  • This feature has some redundancy PR. I'm still working through the use cases for each of them.

Let me know your thoughts and thanks in advance!

nikhil and others added 3 commits August 26, 2022 16:43
✔️ add count to beaglecli configure. This allows the max page number to be returned if --all is specified on list commands
✔️ add cleanup script to get the metadata formatted nicely
✔️ add column_names file for check and docs update
@buehlere buehlere requested a review from nikhil October 11, 2022 15:33
✔️ adding shabang
@buehlere buehlere changed the title init feature commit Metadata File Oct 12, 2022
@nikhil nikhil merged commit 1d718ea into develop Nov 15, 2022
@nikhil nikhil deleted the feature/list_all_metadata branch November 15, 2022 16:09
@nikhil nikhil mentioned this pull request Nov 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants