Skip to content

update docs#17

Merged
luca-chen198 merged 3 commits into
nodes-app:mainfrom
xandaaaa:contributing-page
May 13, 2026
Merged

update docs#17
luca-chen198 merged 3 commits into
nodes-app:mainfrom
xandaaaa:contributing-page

Conversation

@xandaaaa
Copy link
Copy Markdown
Collaborator

No description provided.

@Nicolas-Py
Copy link
Copy Markdown
Collaborator

also maybe change the PR section

Branch from main
Keep the change focused — one logical change per PR
Include test coverage for new behavior
Update CHANGELOG.md under [Unreleased] with a one-line summary
Update DocC docs for any public-API change
Make sure swift build and swift test are green locally before opening the PR; CI will run the same checks

I think only we as members can actually branch off main. Non members need to fork for a future pr afaik. But please confirm @xandaaaa

@xandaaaa
Copy link
Copy Markdown
Collaborator Author

@Nicolas-Py You are totally right. I went through all the docs and cleaned them up now

@xandaaaa xandaaaa changed the title update contributing md update docs May 13, 2026
Copy link
Copy Markdown
Collaborator

@luca-chen198 luca-chen198 left a comment

Choose a reason for hiding this comment

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

gj

@luca-chen198 luca-chen198 merged commit 6244b6a into nodes-app:main May 13, 2026
1 check passed
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.

3 participants