Skip to content

Address peer review#81

Merged
justinmak08 merged 3 commits intodevfrom
address_peer_review
Feb 2, 2026
Merged

Address peer review#81
justinmak08 merged 3 commits intodevfrom
address_peer_review

Conversation

@Olytics
Copy link
Collaborator

@Olytics Olytics commented Feb 2, 2026

This PR adds GitHub profile URLs for all contributors in the CONTRIBUTING file. It also fixes minor issues in the README and applies adjustments to the Quarto configuration and documentation build.

@netlify
Copy link

netlify bot commented Feb 2, 2026

Deploy Preview for flourishing-cajeta-b1d152 ready!

Name Link
🔨 Latest commit 0694f32
🔍 Latest deploy log https://app.netlify.com/projects/flourishing-cajeta-b1d152/deploys/69803d9cf74d7200083aac23
😎 Deploy Preview https://deploy-preview-81--flourishing-cajeta-b1d152.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@codecov
Copy link

codecov bot commented Feb 2, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.23%. Comparing base (00fc1d4) to head (85c8550).
⚠️ Report is 24 commits behind head on dev.

Additional details and impacted files
@@             Coverage Diff             @@
##              dev      #81       +/-   ##
===========================================
+ Coverage   58.65%   94.23%   +35.57%     
===========================================
  Files           6        6               
  Lines         104      104               
  Branches       39       39               
===========================================
+ Hits           61       98       +37     
+ Misses         43        3       -40     
- Partials        0        3        +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Olytics Olytics requested a review from EricYangg February 2, 2026 06:02
@Olytics Olytics self-assigned this Feb 2, 2026
@EricYangg EricYangg changed the base branch from main to dev February 2, 2026 06:07
Copilot AI review requested due to automatic review settings February 2, 2026 23:17
Copy link

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 addresses peer review feedback by adding GitHub profile URLs for contributors and fixing documentation issues. The changes improve documentation clarity and maintainability.

Changes:

  • Added GitHub profile URL link to project name in README
  • Updated contributor information in CONTRIBUTING file
  • Applied adjustments to Quarto configuration and documentation build

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

@justinmak08 justinmak08 merged commit 193d371 into dev Feb 2, 2026
11 checks passed
@justinmak08 justinmak08 deleted the address_peer_review branch February 4, 2026 01:16
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