Skip to content

Update view source link to link to the related page's source code on GitHub#2

Merged
bmurty merged 8 commits intomainfrom
copilot/update-view-source-link
Dec 2, 2025
Merged

Update view source link to link to the related page's source code on GitHub#2
bmurty merged 8 commits intomainfrom
copilot/update-view-source-link

Conversation

Copy link
Contributor

Copilot AI commented Dec 1, 2025

  • Explore and understand the repository structure
  • Identify the "View source" link in src/layouts/page.layout.njk (line 106-110)
  • Understand that Lume provides page.src.path and page.src.ext for the source file path
  • Update the "View source" link to dynamically link to the specific page's source file on GitHub
  • Run linting to verify the change
  • Run code review
  • Run codeql_checker (no issues found)
  • Restore accidentally deleted deno.lock file
  • Simplify title attribute to "View source"
Original prompt

Update view source link to link to the related page's source code on GitHub


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI self-assigned this Dec 1, 2025
Copilot AI and others added 2 commits December 1, 2025 22:38
Co-authored-by: brendanmurty <608191+brendanmurty@users.noreply.github.com>
Co-authored-by: brendanmurty <608191+brendanmurty@users.noreply.github.com>
Copy link
Owner

Choose a reason for hiding this comment

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

@copilot this file should not have been deleted, restore it

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Restored the deno.lock file in commit fe57d45.

Copilot AI changed the title [WIP] Update view source link to GitHub for related pages Update view source link to link to the related page's source code on GitHub Dec 1, 2025
Copilot AI requested a review from bmurty December 1, 2025 22:45
Co-authored-by: brendanmurty <608191+brendanmurty@users.noreply.github.com>
href="https://github.com/brendanmurty/site"
title="View this website's code via the public GitHub repository">
href="https://github.com/brendanmurty/site{% if page.src %}/blob/main/content{{ page.src.path }}{{ page.src.ext }}{% endif %}"
title="View the source code for this page on GitHub">
Copy link
Owner

Choose a reason for hiding this comment

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

@copilot Change the title content to View source instead

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Changed the title attribute to "View source" in commit 20c60df.

Co-authored-by: brendanmurty <608191+brendanmurty@users.noreply.github.com>
Copilot AI requested a review from bmurty December 1, 2025 23:21
@bmurty
Copy link
Owner

bmurty commented Dec 2, 2025

@copilot Merge in the main branch again

Copy link
Contributor Author

Copilot AI commented Dec 2, 2025

@copilot Merge in the main branch again

I'm unable to merge branches from GitHub. Please merge the main branch into this PR manually, or I can help resolve any conflicts after you've initiated the merge.

@bmurty bmurty marked this pull request as ready for review December 2, 2025 01:27
@bmurty bmurty merged commit a20dc13 into main Dec 2, 2025
1 check passed
@bmurty bmurty deleted the copilot/update-view-source-link branch December 2, 2025 01:27
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