Skip to content

Conversation

@JR-1991
Copy link
Member

@JR-1991 JR-1991 commented Dec 2, 2023

  • Added tests for the dataverse action
    • GET - api/info/version
    • GET - api/metadatablocks
    • POST - api/dataverses/root
  • Removed redundant files and steps
    • unused scripts
    • action steps that call scripts
  • Updated README.md and added tests badge

Successful action run

@JR-1991 JR-1991 added documentation Improvements or additions to documentation enhancement New feature or request labels Dec 2, 2023
@JR-1991 JR-1991 requested a review from poikilotherm December 2, 2023 00:02
@JR-1991 JR-1991 self-assigned this Dec 2, 2023
@JR-1991 JR-1991 requested a review from pdurbin December 2, 2023 00:02
Copy link
Member

@poikilotherm poikilotherm left a comment

Choose a reason for hiding this comment

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

Looks good to me. Just one small suggestion, as I think the Syntax fornusing the env variable in the run example is not valid.

@JR-1991
Copy link
Member Author

JR-1991 commented Dec 2, 2023

@poikilotherm, thank you for your feedback and for pointing out the example. Commited your proposed changes to the PR.

I have also extended the outputs of the action to expose the Dataverse version. PyDataverse tests had the version hard-coded, but now with this addition, multiple versions can be tested. 🙌

@poikilotherm
Copy link
Member

Thank you @JR-1991 for the PR, just merged it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants