Skip to content

Installable version of client tools #5

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.

Sign up for GitHub

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

Jump to bottom
Draft
rdhyee wants to merge 25 commits into
base: master
Choose a base branch
from
Draft

Installable version of client tools #5

rdhyee wants to merge 25 commits into from

Conversation

@rdhyee
Copy link

@rdhyee rdhyee commented Jun 23, 2023 •
edited
Loading

Some goals of this PR:

  • to provide an pip-installable Python client for the EZID API (v2)
  • to write code that is build upon the prior work of CDLUC3 in the hopes that CDL will maintain this rewrite of the client

The main work in this PR is copying https://github.com/CDLUC3/ezid/blob/main/tools/client.py to this repository and then rewriting client.py to have both a Client class (that can easily be called from other Python code) and a ConsoleClient class that preserves the original command line functionality of ezid.py. Client is a light reworking of the command line syntax.

rdhyee added 25 commits May 9, 2023 09:37
@rdhyee
an attempt to get flit build to work -- had to commit stuff
6498364
@rdhyee
Wrap commandline operations in ezid3.py and batch-register3.py around…
6917854 
… main() and invoke main only for commandline operations (not imports)
@rdhyee
Now ezid3 and batch-register3 installs
65c6250
@rdhyee
ezid and batch-register are now installed properly.
724b7d9 
ezid handles different servers.
@rdhyee
transition to client.py, code from https://github.com/CDLUC3/ezid/blo…
d276f36 
…b/main/tools/client.py
@rdhyee
have a Client and a ConsoleClient -- ConsoleClient sends output to st…
51bb05d 
…dout
@rdhyee
cleaning up the ConsoleClient.__getattribute__ call to get rid of an …
013c5ab 
…explicit mention of ConsoleClient
@rdhyee
update the staging EZID server URL
8c3e966 
separate out the parser generator into a separate method and allow for a default args for Client
@rdhyee
changing methods of utils.ANVL to static methods
75352d5
@rdhyee
client.Client was throwing sys exits -- move that sys exception to th…
d8db1dd 
…e ConsoleClient
@rdhyee
surfacing headers and status to the Client
af02c59
@rdhyee
in parsing anvl, handle the case in which there might be colons in th…
4305ead 
…e metadata
@rdhyee
first commit for tests.py
71c3220
@rdhyee
Add TestConsoleClient
ae019eb
@rdhyee
Create python-package.yml
6792ad3 
@rdhyee is new to GitHub actions -- so this PR is an attempt to get this configured to run tests.
@rdhyee
Merge pull request #1 from rdhyee/add-python-package-yml
754134a 
Create python-package.yml
@rdhyee
Merge remote-tracking branch 'origin' into installable
ca83f8a
@rdhyee
Reorganizing files -- moving module to src and added a version.py
1e3f225
@rdhyee
reworking code to make flake8 happy
e6cd6bd
@rdhyee
do a pass in which we import ezid_client_tools by first putting the m…
1015fe0 
…odule in sys.path
@rdhyee
Let's see whether pytest can pick up the EZID credentials
3b8b6c7
@rdhyee
add pip install and remove the sys.path manipulation in test_client.py
823b8ba
@rdhyee
pull version number out to separate file and add build_and_release Gi…
5054e82 
…tHub action
@rdhyee
try again in terms of reading in VERSION
c09c5c9
@rdhyee
fixing build process because this is a pure Python build
1a8fd47
@jsjiang jsjiang mentioned this pull request Jun 28, 2023
Open
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.

1 participant

@rdhyee