Skip to content

Add installation instructions in README#54

Merged
niksirbi merged 5 commits into
mainfrom
add-install-instructions
May 26, 2026
Merged

Add installation instructions in README#54
niksirbi merged 5 commits into
mainfrom
add-install-instructions

Conversation

@niksirbi

Copy link
Copy Markdown
Member

Description

What is this PR

  • Bug fix
  • Addition of a new feature
  • Other

Why is this PR needed?

I realised we don't have any sort of installation instructions. This was mostly fine, since the package is not yet on PyPI and probably has no users other than us, but that's about to change. We're planning to share this repo with collaborators soon, who will try to use it to contribute data.

What does this PR do?

  • Adds some minimal installation instructions in the README, for creating a venv with uv and installing the package directly from GitHub
  • Fixes a broken link to the contributing guide.

It DOES NOT add an installation guide to the docs; That will follow after we publish the package on PyPI.

References

N/A

How has this PR been tested?

I reproduced the installation steps locally.

Is this a breaking change?

No.

Does this PR require an update to the documentation?

N/A

Checklist:

  • The code has been tested locally
  • Tests have been added to cover all new functionality
  • The documentation has been updated to reflect any changes
  • The code has been formatted with pre-commit

@niksirbi
niksirbi force-pushed the add-install-instructions branch from 363d647 to 7af491f Compare May 26, 2026 17:11
@niksirbi
niksirbi marked this pull request as ready for review May 26, 2026 17:15
@niksirbi
niksirbi requested a review from a team May 26, 2026 17:15

@sfmig sfmig left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, just a small comment

Comment thread README.md Outdated
niksirbi and others added 2 commits May 26, 2026 18:59
Co-authored-by: sfmig <33267254+sfmig@users.noreply.github.com>
@niksirbi

Copy link
Copy Markdown
Member Author

Thanks @sfmig! Good call about explicitly specifying the branch. I've added that and will merge as soon as CI passes.

@niksirbi
niksirbi merged commit 24e4a37 into main May 26, 2026
24 checks passed
@niksirbi
niksirbi deleted the add-install-instructions branch May 26, 2026 18:05
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.

2 participants