Skip to content

Link to Computational Physics Group website#1168

Merged
sbryngelson merged 1 commit intomasterfrom
group-site-links
Feb 19, 2026
Merged

Link to Computational Physics Group website#1168
sbryngelson merged 1 commit intomasterfrom
group-site-links

Conversation

@sbryngelson
Copy link
Member

@sbryngelson sbryngelson commented Feb 19, 2026

User description

Summary

  • Add group website link to README contact section
  • Add group website link to docs landing page footer (amber accent, next to copyright)

These are static links — zero maintenance.

Test plan

  • CI passes
  • README renders correctly on GitHub
  • Docs footer shows link after next deploy

🤖 Generated with Claude Code


CodeAnt-AI Description

Add Computational Physics Group link to README and docs footer

What Changed

  • README contact section now displays a direct link to the Computational Physics Group website next to the project maintainers.
  • Documentation landing page footer shows a visible, amber-accented link to the Computational Physics Group at Georgia Tech.

Impact

✅ Easier to find the Computational Physics Group website
✅ Clearer project affiliation and attribution
✅ Faster access to group contact and collaborator information

💡 Usage Guide

Checking Your Pull Request

Every time you make a pull request, our system automatically looks through it. We check for security issues, mistakes in how you're setting up your infrastructure, and common code problems. We do this to make sure your changes are solid and won't cause any trouble later.

Talking to CodeAnt AI

Got a question or need a hand with something in your pull request? You can easily get in touch with CodeAnt AI right here. Just type the following in a comment on your pull request, and replace "Your question here" with whatever you want to ask:

@codeant-ai ask: Your question here

This lets you have a chat with CodeAnt AI about your pull request, making it easier to understand and improve your code.

Example

@codeant-ai ask: Can you suggest a safer alternative to storing this secret?

Preserve Org Learnings with CodeAnt

You can record team preferences so CodeAnt AI applies them in future reviews. Reply directly to the specific CodeAnt AI suggestion (in the same thread) and replace "Your feedback here" with your input:

@codeant-ai: Your feedback here

This helps CodeAnt AI learn and adapt to your team's coding style and standards.

Example

@codeant-ai: Do not flag unused imports.

Retrigger review

Ask CodeAnt AI to review the PR again, by typing:

@codeant-ai: review

Check Your Repository Health

To analyze the health of your code repository, visit our dashboard at https://app.codeant.ai. This tool helps you identify potential issues and areas for improvement in your codebase, ensuring your repository maintains high standards of code health.

Summary by CodeRabbit

  • Documentation
    • Added attribution to the Computational Physics Group at Georgia Tech and collaborators
    • Added contact information and links to the Slack channel and website
    • Added footer link to the Computational Physics Group @ Georgia Tech

Add group website link to README contact section and docs
landing page footer for better discoverability.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings February 19, 2026 10:41
@codeant-ai
Copy link
Contributor

codeant-ai bot commented Feb 19, 2026

CodeAnt AI is reviewing your PR.


Thanks for using CodeAnt! 🎉

We're free for open-source projects. if you're enjoying it, help us grow by sharing.

Share on X ·
Reddit ·
LinkedIn

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 19, 2026

📝 Walkthrough

Walkthrough

Added attribution and contact information to the README.md Getting Started section, including affiliation with Computational Physics Group at Georgia Tech and contact details for Spencer Bryngelson. Also added a corresponding hyperlink to the Computational Physics Group website in the documentation footer.

Changes

Cohort / File(s) Summary
Documentation Attribution
README.md
Added new paragraph with institutional affiliation, contact information for Spencer Bryngelson, and references to Slack channel and website.
Footer Navigation
docs/index.html
Added hyperlink to Computational Physics Group @ Georgia Tech website in the footer next to copyright line.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested labels

Review effort 2/5

Poem

🐰 A link hops forth with pride so true,
To Georgia Tech's physics crew,
Attribution added, clear and bright,
Contact info set just right! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The pull request description lacks required sections (Type of change, Testing, Checklist) specified in the repository's template. Add the missing template sections: select the Type of change (Documentation), provide details in the Testing section about how changes were verified, and complete the Checklist items.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: adding links to the Computational Physics Group website to the README and docs footer.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch group-site-links

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@codeant-ai codeant-ai bot added the size:XS This PR changes 0-9 lines, ignoring generated files label Feb 19, 2026
@codeant-ai
Copy link
Contributor

codeant-ai bot commented Feb 19, 2026

Sequence Diagram

This PR adds a static link to the Computational Physics Group website in the README contact section and the docs landing page footer. The diagram shows the high-level flow from the developer's change through CI and deployment to the rendered README and docs site.

sequenceDiagram
    participant Developer
    participant Repository
    participant CI
    participant GitHubPages
    participant User

    Developer->>Repository: Create PR modifying README.md and docs/index.html (add group link)
    Repository->>CI: Trigger CI build for PR
    CI-->>Repository: Build & tests pass
    Repository->>GitHubPages: Merge & deploy documentation
    GitHubPages-->>User: Docs landing shows footer with Computational Physics Group link
    Repository-->>User: GitHub README renders with Computational Physics Group link
Loading

Generated by CodeAnt AI

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 2 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Copy link
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

This PR adds static links to the Computational Physics Group website to improve visibility and attribution. The links are placed in high-visibility locations (README contact section and docs landing page footer) with zero maintenance overhead.

Changes:

  • Add group website link to README contact section with contextual text about MFC development
  • Add styled link to docs footer with amber accent color next to the existing copyright

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
README.md Adds a new line crediting the Computational Physics Group at Georgia Tech as MFC developers with a hyperlink
docs/index.html Adds a second link in the footer to the Computational Physics Group website with amber styling

@codeant-ai
Copy link
Contributor

codeant-ai bot commented Feb 19, 2026

Nitpicks 🔍

🔒 No security issues identified
⚡ Recommended areas for review

  • External Link Attributes
    The new anchor for the Computational Physics Group lacks explicit external-link behavior attributes (e.g., target and rel). For outbound links it's best to decide consistent UX (same tab vs new tab) and, when opening a new tab, include rel="noopener noreferrer" to avoid reverse tabnapping and protect the site.

  • Accessibility / Discoverability
    The added link text is clear, but consider adding an accessible hint or consistent visual cue for external links (aria-label or an icon) so screen reader users know it's an external destination and maintain consistent affordance across other footer links.

@codeant-ai
Copy link
Contributor

codeant-ai bot commented Feb 19, 2026

CodeAnt AI finished reviewing your PR.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

🧹 Nitpick comments (1)
docs/index.html (1)

250-252: Duplicate link destination in footer — consider consolidating.

Lines 250 and 251 both link to https://comp-physics.group/: the existing copyright anchor text ("Spencer H. Bryngelson and Tim Colonius") and the new explicit group label. Adjacent links to the same URL are redundant and can confuse screen readers, which will announce two separate link targets pointing to the same page.

Additionally, the existing copyright <a> at line 250 has no styling class (renders as undecorated white text), while the new link at line 251 uses text-amber-400 hover:underline, creating a visual inconsistency within the same line.

Consider removing the hyperlink from the copyright names and keeping only the new descriptive link:

♻️ Proposed consolidation
-            &copy; <span id="copyright-year"></span> <a href="https://comp-physics.group/">Spencer H. Bryngelson and Tim Colonius</a>
-            &middot; <a href="https://comp-physics.group/" class="text-amber-400 hover:underline">Computational Physics Group @ Georgia Tech</a>
+            &copy; <span id="copyright-year"></span> Spencer H. Bryngelson and Tim Colonius
+            &middot; <a href="https://comp-physics.group/" class="text-amber-400 hover:underline">Computational Physics Group @ Georgia Tech</a>
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@docs/index.html` around lines 250 - 252, Consolidate the duplicate footer
links by removing the hyperlink from the "Spencer H. Bryngelson and Tim
Colonius" anchor and leaving the styled "Computational Physics Group @ Georgia
Tech" link as the single link target; locate the elements around the span with
id="copyright-year" and replace or adjust the first <a> (text "Spencer H.
Bryngelson and Tim Colonius") so it is plain text (no href/class) while keeping
the second <a> with class="text-amber-400 hover:underline" as the accessible,
styled link to https://comp-physics.group/.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@docs/index.html`:
- Around line 250-252: Consolidate the duplicate footer links by removing the
hyperlink from the "Spencer H. Bryngelson and Tim Colonius" anchor and leaving
the styled "Computational Physics Group @ Georgia Tech" link as the single link
target; locate the elements around the span with id="copyright-year" and replace
or adjust the first <a> (text "Spencer H. Bryngelson and Tim Colonius") so it is
plain text (no href/class) while keeping the second <a> with
class="text-amber-400 hover:underline" as the accessible, styled link to
https://comp-physics.group/.

@sbryngelson sbryngelson merged commit 05b26f7 into master Feb 19, 2026
35 checks passed
@sbryngelson sbryngelson deleted the group-site-links branch February 19, 2026 10:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS This PR changes 0-9 lines, ignoring generated files

Development

Successfully merging this pull request may close these issues.

1 participant

Comments