Skip to content

Fix broken STYLE_GUIDE.md references#1068

Merged
jacobsimionato merged 6 commits intogoogle:mainfrom
selamw1:fix-docs-style-guide-links
Apr 14, 2026
Merged

Fix broken STYLE_GUIDE.md references#1068
jacobsimionato merged 6 commits intogoogle:mainfrom
selamw1:fix-docs-style-guide-links

Conversation

@selamw1
Copy link
Copy Markdown
Collaborator

@selamw1 selamw1 commented Apr 3, 2026

This PR addresses issue #1028 where references to STYLE_GUIDE.md are broken because the file does not exist.

The chosen solution is to remove these references and link to external Google Style Guides or relevant sections in CONTRIBUTING.md.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the documentation by replacing internal style guide references with links to external Google style guides in the pull request template and contribution guidelines. Feedback suggests refining the terminology for the guidelines list to be more inclusive of project-specific requirements and providing a direct link to the gts repository for TypeScript contributors.

Comment thread CONTRIBUTING.md Outdated
Comment thread CONTRIBUTING.md Outdated
selamw1 and others added 5 commits April 3, 2026 13:22
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@jacobsimionato jacobsimionato merged commit f42f2dc into google:main Apr 14, 2026
9 checks passed
@github-project-automation github-project-automation Bot moved this from Todo to Done in A2UI Apr 14, 2026
@selamw1 selamw1 deleted the fix-docs-style-guide-links branch April 14, 2026 22:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants