Skip to content

[Contract] "Source Code" to allow lowercase README.md#2046

Merged
jeesunikim merged 4 commits intomainfrom
source-code-enable-lowercase-readme
Apr 18, 2026
Merged

[Contract] "Source Code" to allow lowercase README.md#2046
jeesunikim merged 4 commits intomainfrom
source-code-enable-lowercase-readme

Conversation

@jeesunikim
Copy link
Copy Markdown
Contributor

Screenshot 2026-04-17 at 1 40 12 PM

Copilot AI review requested due to automatic review settings April 17, 2026 20:40
@github-project-automation github-project-automation Bot moved this to Backlog (Not Ready) in DevX Apr 17, 2026
@jeesunikim jeesunikim requested a review from quietbits April 17, 2026 20:41
Copy link
Copy Markdown

@oceans404 oceans404 left a comment

Choose a reason for hiding this comment

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

Thanks Jeesun!

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the GitHub README fetch used by the “Source Code” view to support repositories whose readme file is named readme.md (lowercase), improving compatibility with differing filename casing in GitHub repos.

Changes:

  • Switches the README fetch response variable to be reassignable.
  • Adds a fallback request to readme.md when README.md is not found.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/query/useGitHubReadmeText.ts Outdated
@stellar-jenkins-ci
Copy link
Copy Markdown

1 similar comment
@stellar-jenkins-ci
Copy link
Copy Markdown

@stellar-jenkins-ci
Copy link
Copy Markdown

@stellar-jenkins-ci
Copy link
Copy Markdown

@jeesunikim jeesunikim merged commit 5b8127c into main Apr 18, 2026
7 checks passed
@jeesunikim jeesunikim deleted the source-code-enable-lowercase-readme branch April 18, 2026 01:33
@github-project-automation github-project-automation Bot moved this from Backlog (Not Ready) to Done in DevX Apr 18, 2026
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.

4 participants