Skip to content

Fix a typo in CONTRIBUTING.md#17195

Merged
AndyScherzinger merged 1 commit into
nextcloud:masterfrom
sitegui:bugfix-contributing-md
Jun 18, 2026
Merged

Fix a typo in CONTRIBUTING.md#17195
AndyScherzinger merged 1 commit into
nextcloud:masterfrom
sitegui:bugfix-contributing-md

Conversation

@sitegui

@sitegui sitegui commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Hello Nextcloud Android team o/

I've started studying the codebase and I've noticed a small typo in the markdown of CONTRIBUTING.md that, causing part of the file to be rendered incorrectly.

🖼️ Screenshots

🏚️ Before

https://github.com/nextcloud/android/blob/448b459165b78f376a7bfb4db9a8562daa310938/CONTRIBUTING.md#testing

grafik

🏡 After

https://github.com/sitegui/nextcloud-android/blob/978b49eb281efca974355c4d197cc868e6c977b1/CONTRIBUTING.md#testing

grafik

🏁 Checklist

  • ⛑️ Tests (unit and/or integration) are included or not needed
  • 🔙 Backport requests are created or not needed: /backport to stable-xx.x
  • 📅 Milestone is set
  • 🌸 PR title is meaningful (if it should be in the changelog: is it meaningful to users?)

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI

The code block had the language "bash" written at the end of the block, causing part of the file to be rendered incorrectly.

Signed-off-by: sitegui <2801575+sitegui@users.noreply.github.com>
@sitegui

sitegui commented Jun 16, 2026

Copy link
Copy Markdown
Contributor Author

Just to give more context: I'm also having problems with the auto-upload feature.

I saw the call for help in #16550, so I would like to use this as an excuse to learn more about the internals of this project and maybe help with the fixes. 🤓

Do you have the auto-upload architecture documented somewhere?
I guess I'll write one for me to help understanding it. I can share it later, if you feel that it's interesting to have it on the codebase.

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

2nd commit 3d2955b needs to be removed as is is not in scope for the PR and it is also a set of false changes we would not except, like the logging flag change as well as the app-id and cmake changes

@AndyScherzinger

Copy link
Copy Markdown
Member

Just to give more context: I'm also having problems with the auto-upload feature.

Best to find an existing ticket or raising a new one to not mix-up different issues in a single Github issue.

The first commit was fine from my pov, but the 2nd commit makes it a blocker.

@sitegui sitegui force-pushed the bugfix-contributing-md branch from 3d2955b to 978b49e Compare June 18, 2026 09:28
@sitegui

sitegui commented Jun 18, 2026

Copy link
Copy Markdown
Contributor Author

Thanks @AndyScherzinger !

Sorry for the second commit, it was a mistake. I think it was auto-pushed when I ran ./script/buildDev.

@AndyScherzinger AndyScherzinger merged commit d1c43d4 into nextcloud:master Jun 18, 2026
17 of 23 checks passed
@AndyScherzinger

Copy link
Copy Markdown
Member

No worries @sitegui and thanks you very much for the fix! 🙏 Merged 🚀

@sitegui sitegui deleted the bugfix-contributing-md branch June 18, 2026 15:38
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