Skip to content

Add developer documentation#36

Merged
K-Meech merged 7 commits into
mainfrom
km/dev-docs
Jun 1, 2026
Merged

Add developer documentation#36
K-Meech merged 7 commits into
mainfrom
km/dev-docs

Conversation

@K-Meech

@K-Meech K-Meech commented Jun 1, 2026

Copy link
Copy Markdown
Collaborator

Description

What is this PR

  • Bug fix
  • Addition of a new feature
  • Other

Why is this PR needed?
Currently, there are no docs for developer setup

What does this PR do?

  • Adds documentation about how to setup a development environment for OSCaR
  • Removes placeholder text from the docs template
  • Moves docs dependencies into pyproject.toml, so they're all in one place

I'll add a link to the built github pages site, once I've set that up in the project settings.

References

Closes #35

How has this PR been tested?

N/A - only docs changes

Is this a breaking change?

No

Does this PR require an update to the documentation?

Docs have been updated.

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

@K-Meech K-Meech requested a review from adamltyson June 1, 2026 12:30

@adamltyson adamltyson 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 couple of tiny suggestions

Comment thread docs/source/conf.py Outdated
Comment thread docs/source/conf.py
"name": "GitHub",
# URL where the link will redirect
"url": "provide later", # required
"url": "https://github.com/neuroinformatics-unit/OSCaR",

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.

oscar.neuroinformatics.dev?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I'll use this domain when I setup the github pages deployment 👍
This line refers to the link for the github logo at the top right of the website - so I'll leave that one as-is.

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.

ha, good spot, thanks!

@K-Meech K-Meech merged commit 479fef3 into main Jun 1, 2026
12 checks passed
@K-Meech K-Meech deleted the km/dev-docs branch June 1, 2026 15:33
@K-Meech K-Meech mentioned this pull request Jun 1, 2026
7 tasks
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.

Add developer setup docs

2 participants