Skip to content

Add contributing guidelines#118

Merged
egrace479 merged 7 commits into
devfrom
feat/contributing
Jul 8, 2026
Merged

Add contributing guidelines#118
egrace479 merged 7 commits into
devfrom
feat/contributing

Conversation

@egrace479

Copy link
Copy Markdown
Member

The goal is to be open and encouraging.

@egrace479 egrace479 requested review from beanbean9339 and hlapp July 8, 2026 18:36
@egrace479 egrace479 added the documentation Improvements or additions to documentation label Jul 8, 2026
Comment thread CONTRIBUTING.md Outdated
Comment thread CONTRIBUTING.md Outdated

@beanbean9339 beanbean9339 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good overall! The contribution guidelines are clear and cover the main contributor workflow, testing expectations, and repository conventions. I only noticed a small grammar typo ("Included tests and documentation updates..." → "Include tests and documentation updates..."), but otherwise everything looks good.

@hlapp hlapp 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.

See inline suggestions. There's one though where it wasn't immediately clear what to suggest, hence the change request.

Comment thread CONTRIBUTING.md Outdated
Comment thread CONTRIBUTING.md Outdated
Comment thread CONTRIBUTING.md Outdated
Comment thread CONTRIBUTING.md Outdated
Comment thread CONTRIBUTING.md Outdated
hlapp and others added 2 commits July 8, 2026 15:44
Co-authored-by: Elizabeth Campolongo <38985481+egrace479@users.noreply.github.com>
Co-authored-by: Hilmar Lapp <hlapp@drycafe.net>
@egrace479 egrace479 requested a review from hlapp July 8, 2026 19:46

@hlapp hlapp 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.

👍🏻

@egrace479 egrace479 merged commit dbfabf7 into dev Jul 8, 2026
1 check passed
@egrace479 egrace479 deleted the feat/contributing branch July 8, 2026 19:53
egrace479 added a commit that referenced this pull request Jul 8, 2026
* Add description of contributing guidelines

* Clarify default setting of repo as Imageomics Catalog

Co-authored-by: Isabella Lo <isabellalo9339@gmail.com>

* Add note about contributing to the template

* Apply suggestions from code review

Co-authored-by: Hilmar Lapp <hlapp@drycafe.net>

---------

Co-authored-by: Isabella Lo <isabellalo9339@gmail.com>
Co-authored-by: Hilmar Lapp <hlapp@drycafe.net>
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants