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

Admin interface #166

Open
wants to merge 41 commits into
base: develop
Choose a base branch
from
Open

Admin interface #166

wants to merge 41 commits into from

Conversation

bwbrock
Copy link

@bwbrock bwbrock commented Nov 30, 2023

Any change needs to be discussed before proceeding. Failure to do so may result in the rejection of the pull request.

Thanks for submitting a pull request! It's great, that you want contribute to pyDataverse. Please provide enough information so that others can review it.

First, start always by reading the Contribution Guide. There you can find all information needed, to create good pull requests.

All Submissions

Describe your environment

  • OS: NAME, VERSION, 64/32bit
  • pyDataverse: VERSION
  • Python: VERSION
  • Dataverse: VERSION

Follow best practices

  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • Have you followed the guidelines in our Contribution Guide?
  • Have you read the Code of Conduct?
  • Do your changes in a seperate branch. Branches MUST have descriptive names.
  • Have you merged the latest changes from upstream to your branch?

Describe the PR

  • What kind of change does this PR introduce?
    • TEXT
  • Why is this change required? What problem does it solve?
    • TEXT
  • Screenshots (if appropriate)
  • Put Closes #ISSUE_NUMBER to the end of this pull request

Testing

  • Have you used tox and/or pytest for testing the changes?
  • Did the local testing ran successfully?
  • Did the Continous Integration testing (Travis-CI) ran successfully?

Commits

  • Have descriptive commit messages with a short title (first line).
  • Use the commit message template
  • Put Closes #ISSUE_NUMBER in your commit messages to auto-close the issue that it fixes (if such).

Others

  • Is there anything you need from someone else?

Documentation contribution

  • Have you followed NumPy Docstring standard?

Code contribution

  • Have you used pre-commit?
  • Have you formatted your code with black prior to submission (e. g. via pre-commit)?
  • Have you written new tests for your changes?
  • Have you ran mypy on your changes successfully?
  • Have you documented your update (Docstrings and/or Docs)?
  • Do your changes require additional changes to the documentation?

Closes #ISSUE_NUMBER

@bwbrock bwbrock marked this pull request as ready for review November 30, 2023 19:51
@bwbrock
Copy link
Author

bwbrock commented Dec 6, 2023

I'm catching up with Jan before working on this further. I probably need to refactor or add tests to fit the CI practices for pyDataverse.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

2 participants