Skip to content

Add collaboration guideline (AlphaGenes/AlphaPeel#223)#224

Merged
XingerTang merged 1 commit into
AlphaGenes:develfrom
XingerTang:doc/issue-223-add-collab-guide
Mar 6, 2026
Merged

Add collaboration guideline (AlphaGenes/AlphaPeel#223)#224
XingerTang merged 1 commit into
AlphaGenes:develfrom
XingerTang:doc/issue-223-add-collab-guide

Conversation

@XingerTang

Copy link
Copy Markdown
Contributor

Related Issue

Closes #223

What changed

Add a section of the collaboration guide in addition to the original technical guide of the contribution development guide

Why this change

See #223

Notes / Risks

@XingerTang XingerTang requested a review from gregorgorjanc March 2, 2026 16:50
@gregorgorjanc

Copy link
Copy Markdown
Member

@XingerTang can you have a look if there are any useful bits that you could take from https://github.com/HighlanderLab/RcppTskit/pull/119/changes#diff-9e530a148d1cbfcb4690a1c1f9aa295b6bd1728bc12afe3c762f56b66f269f47? Can you also suggest me, what are the useful bits I can take from you here to RcppTskit dev guide!

Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst
Comment thread docs/source/contribute.rst
Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst
Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst
Comment thread docs/source/contribute.rst
Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst Outdated
Comment thread docs/source/contribute.rst Outdated

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

@XingerTang I have gone over and left some suggested edits. Apply these and merge this in!

@XingerTang

Copy link
Copy Markdown
Contributor Author

@XingerTang I have gone over and left some suggested edits. Apply these and merge this in!

Thank you @gregorgorjanc! I will soon apply the changes. There is only one point that the emojis are taking up more space than they are displayed. If I reduce the separators to exactly what the line length shows, the renderer would complain.

@gregorgorjanc

Copy link
Copy Markdown
Member

Then maybe opt for simple ASCII numbers;)

@XingerTang XingerTang force-pushed the doc/issue-223-add-collab-guide branch from d7a1723 to f7e5336 Compare March 6, 2026 13:05
@XingerTang XingerTang force-pushed the doc/issue-223-add-collab-guide branch from f7e5336 to ca1de97 Compare March 6, 2026 13:08
@XingerTang XingerTang merged commit 93526cc into AlphaGenes:devel Mar 6, 2026
3 checks passed
@XingerTang XingerTang deleted the doc/issue-223-add-collab-guide branch March 10, 2026 15:19
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