Skip to content

docs: refresh README - #374

Merged
ChiragAgg5k merged 3 commits into
masterfrom
dev
Oct 17, 2026
Merged

ChiragAgg5k merged 3 commits into
masterfrom
dev

Conversation

@ChiragAgg5k

Copy link
Copy Markdown
Member

Refreshes the README with the rewritten getting-started guide and corrected badges, links and install steps from appwrite/sdk-generator#2077 and appwrite/appwrite#14613.

README files only. No code, version or changelog changes, so this does not need a release.

@hansi-codes

hansi-codes Bot commented Oct 17, 2026 •

Copy link
Copy Markdown

🟢 Tier S · Ready to merge

The README replaces the older SDK overview with refreshed installation, authentication, resource-management, local-development, code-generation, and CI/CD guides. It also updates badges, contribution instructions, and the license link.

Latest changes: The newest commits update the push all resource list to include analytics and add Python to the documented types --language options.

Verdict New comments Fixed Still open
✅ Approved 0 0 0
📂 Walkthrough · 1
File Change
README.md Refreshes CLI installation and usage guidance, adds project workflows and CI/CD examples, and updates links and badges.
🔇 Filtered out · 1

Findings Hansi considered but did not post.

Finding Why
Explain how to trust a self-signed endpoint before login Verifier: The claimed missing-account restriction is handled: internal/cmd/client.go:112-114 creates a default session when none exists, then lines 128-129 store selfSigned on it. A targeted test starting with empty preferences confirmed that client --self-signed true saves the setting and passes login's endpoint verification against a self-signed TLS server. Adding --endpoint is not required for that precheck. There is a separate existing issue in loginWithPassword: its client is constructed without SetSelfSigned and its new session does not inherit the setting, so the proposed README setup order does not make self-signed login work.

Reviewed the commits since 3bbeb1e · Details · Comment @hansi-codes review to re-run, or mention @hansi-codes with a question.

@hansi-codes hansi-codes Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🔵 Tier A · Looks good to merge. Summary

Comment thread README.md Outdated
Comment thread README.md Outdated

@hansi-codes hansi-codes Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Tier S · Looks good to merge. Summary

@ChiragAgg5k
ChiragAgg5k merged commit 60c330f into master Oct 17, 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.

1 participant