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

agents: improve the CLI #68

Merged
merged 1 commit into from
Oct 7, 2021
Merged

Conversation

rerowep
Copy link
Contributor

@rerowep rerowep commented Sep 29, 2021

No description provided.

@rerowep rerowep self-assigned this Sep 29, 2021
@rerowep rerowep marked this pull request as draft September 29, 2021 12:54
@rerowep rerowep force-pushed the wep-#2286-better-cli branch 2 times, most recently from 72f8ceb to a9681d3 Compare September 29, 2021 19:09
@rerowep rerowep marked this pull request as ready for review September 29, 2021 19:43
@rerowep rerowep changed the title Wep #2286 better cli better cli Sep 29, 2021
Copy link

@iGormilhit iGormilhit left a comment

Choose a reason for hiding this comment

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

Commit message proposition:

agents: improve the CLI

* Splits the large `bulk_load` function into smaller functions.

@rerowep rerowep force-pushed the wep-#2286-better-cli branch 5 times, most recently from 3e6049b to 07c2e45 Compare September 30, 2021 09:50
@rerowep rerowep requested a review from iGormilhit September 30, 2021 11:05
@rerowep rerowep force-pushed the wep-#2286-better-cli branch 2 times, most recently from 9a15be3 to 5d5409c Compare September 30, 2021 21:36
@iGormilhit iGormilhit changed the title better cli agents: improve the CLI Oct 5, 2021
@rerowep rerowep force-pushed the wep-#2286-better-cli branch 9 times, most recently from aa4dc1a to c743936 Compare October 6, 2021 22:18
@rerowep rerowep force-pushed the wep-#2286-better-cli branch from c743936 to 043b8a6 Compare October 7, 2021 05:58
* Splits the large `bulk_load` function into smaller functions.
* Restricts jsonschema < 4.0.0 because an incompatibility with invenio!
* Updates security dependencies.

Co-Authored-by: Peter Weber <peter.weber@rero.ch>
@rerowep rerowep force-pushed the wep-#2286-better-cli branch from 043b8a6 to df303bd Compare October 7, 2021 06:37
@rerowep rerowep merged commit 7934c3e into rero:dev Oct 7, 2021
@rerowep rerowep deleted the wep-#2286-better-cli branch October 7, 2021 07:11
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.

4 participants