Skip to content

feat: add Gerry to website (INT-332)#105

Merged
glaracuente merged 3 commits intomasterfrom
feature/INT-332/add-gerry-to-website
Mar 31, 2026
Merged

feat: add Gerry to website (INT-332)#105
glaracuente merged 3 commits intomasterfrom
feature/INT-332/add-gerry-to-website

Conversation

@glaracuente
Copy link
Copy Markdown
Contributor

@glaracuente glaracuente commented Mar 31, 2026

what

  • Add Gerry (self) to the team section of the company website

why

  • I recently joined the company
  • I intentionally chose a weight of 7 instead 5 to leave room for Robin who joined before me =]

references

Summary by CodeRabbit

  • New Features
    • Added a new team member profile to the public team roster, including name, role, bio, profile image, and links to social/professional profiles.

@glaracuente glaracuente requested a review from a team as a code owner March 31, 2026 16:13
@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 31, 2026

Deploy Preview for masterpoint ready!

Name Link
🔨 Latest commit fbc9fff
🔍 Latest deploy log https://app.netlify.com/projects/masterpoint/deploys/69cc19634667230008849d8d
😎 Deploy Preview https://deploy-preview-105--masterpoint.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 31
Accessibility: 86
Best Practices: 83
SEO: 79
PWA: 70
View the detailed breakdown and full score reports

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

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Mar 31, 2026

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (1)
  • static/img/team/gerry-laracuente.png is excluded by !**/*.png

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 49f9db10-f0e9-4341-8a3f-df97789e20b6

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Walkthrough

Added a new team member profile Markdown file content/team/gerry-laracuente.md with front-matter metadata (title, designation, linkedin, github, draft, weight, team, image) and a short bio terminated by a NO UPDATE marker.

Changes

Cohort / File(s) Summary
New Team Profile
content/team/gerry-laracuente.md
Added a new team profile containing front-matter fields (title, designation, linkedin, github, draft: false, weight: 7, team: true, image path) and a brief bio body ending with NO UPDATE.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • oycyc
  • Gowiem
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and specifically summarizes the main change: adding Gerry to the website. It is concise, directly related to the changeset, and includes an issue reference for context.
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 unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/INT-332/add-gerry-to-website

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.

oycyc
oycyc previously approved these changes Mar 31, 2026
Copy link
Copy Markdown
Contributor

@oycyc oycyc left a comment

Choose a reason for hiding this comment

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

👋

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

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@content/team/gerry-laracuente.md`:
- Around line 1-12: Run the formatter and commit the changes so the Prettier
check passes: run `trunk fmt` (or your project's formatting command) on
content/team/gerry-laracuente.md, review the YAML front-matter (fields like
title, designation, linkedin, github, draft, weight, team, image, bio) and the
trailing content ("NO UPDATE"), accept the automated formatting changes and
commit them to the PR so CI succeeds.
- Line 12: Remove the stray template marker text "NO UPDATE" from the team
profile content (the literal string "NO UPDATE" present in the markdown) so it
doesn't render on the published page; simply delete that line or replace it with
the intended profile body text and ensure there are no other leftover template
markers in the file.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 751d7689-8afa-49be-aa44-5e74c23feb5f

📥 Commits

Reviewing files that changed from the base of the PR and between 6d6779e and 09f7c20.

⛔ Files ignored due to path filters (1)
  • static/img/team/gerry-laracuente.png is excluded by !**/*.png
📒 Files selected for processing (1)
  • content/team/gerry-laracuente.md

Comment on lines +1 to +12
---
title: Gerry Laracuente
designation: Platform Engineer
linkedin: https://www.linkedin.com/in/grlaracuente
github: https://github.com/glaracuente
draft: false
weight: 7
team: true
image: /img/team/gerry-laracuente.png
bio: Gerry is a platform engineer from Newark, NJ who recently fulfilled a long-time dream of moving to Puerto Rico. After more than a decade of working with both bare-metal and cloud infrastructure, he's found that his favorite part of this industry is the community. Whether it's helping devs automate away toil or exchanging ideas at DevOps meetups, it's the people that make this career worthwhile. Outside of work, you can catch him spending time with family or being humbled at Jiu Jitsu training.
---
NO UPDATE
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

Prettier check is failing; run formatter before merge.

Static analysis reports formatting failure for this file. Please run trunk fmt so CI passes cleanly.

🧰 Tools
🪛 GitHub Check: Trunk Check

[failure] 1-1: prettier
Incorrect formatting, autoformat by running 'trunk fmt'

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@content/team/gerry-laracuente.md` around lines 1 - 12, Run the formatter and
commit the changes so the Prettier check passes: run `trunk fmt` (or your
project's formatting command) on content/team/gerry-laracuente.md, review the
YAML front-matter (fields like title, designation, linkedin, github, draft,
weight, team, image, bio) and the trailing content ("NO UPDATE"), accept the
automated formatting changes and commit them to the PR so CI succeeds.

image: /img/team/gerry-laracuente.png
bio: Gerry is a platform engineer from Newark, NJ who recently fulfilled a long-time dream of moving to Puerto Rico. After more than a decade of working with both bare-metal and cloud infrastructure, he's found that his favorite part of this industry is the community. Whether it's helping devs automate away toil or exchanging ideas at DevOps meetups, it's the people that make this career worthwhile. Outside of work, you can catch him spending time with family or being humbled at Jiu Jitsu training.
---
NO UPDATE
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

Remove template marker text from published profile content.

NO UPDATE at Line 12 will render as page body text on the team profile and looks unintended.

Suggested fix
 ---
 title: Gerry Laracuente
 designation: Platform Engineer
 linkedin: https://www.linkedin.com/in/grlaracuente
 github: https://github.com/glaracuente
 draft: false
 weight: 7
 team: true
 image: /img/team/gerry-laracuente.png
 bio: Gerry is a platform engineer from Newark, NJ who recently fulfilled a long-time dream of moving to Puerto Rico. After more than a decade of working with both bare-metal and cloud infrastructure, he's found that his favorite part of this industry is the community. Whether it's helping devs automate away toil or exchanging ideas at DevOps meetups, it's the people that make this career worthwhile. Outside of work, you can catch him spending time with family or being humbled at Jiu Jitsu training.
 ---
-NO UPDATE
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
NO UPDATE
---
title: Gerry Laracuente
designation: Platform Engineer
linkedin: https://www.linkedin.com/in/grlaracuente
github: https://github.com/glaracuente
draft: false
weight: 7
team: true
image: /img/team/gerry-laracuente.png
bio: Gerry is a platform engineer from Newark, NJ who recently fulfilled a long-time dream of moving to Puerto Rico. After more than a decade of working with both bare-metal and cloud infrastructure, he's found that his favorite part of this industry is the community. Whether it's helping devs automate away toil or exchanging ideas at DevOps meetups, it's the people that make this career worthwhile. Outside of work, you can catch him spending time with family or being humbled at Jiu Jitsu training.
---
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@content/team/gerry-laracuente.md` at line 12, Remove the stray template
marker text "NO UPDATE" from the team profile content (the literal string "NO
UPDATE" present in the markdown) so it doesn't render on the published page;
simply delete that line or replace it with the intended profile body text and
ensure there are no other leftover template markers in the file.

@oycyc
Copy link
Copy Markdown
Contributor

oycyc commented Mar 31, 2026

Can you run the trunk fmt again

@glaracuente
Copy link
Copy Markdown
Contributor Author

Can you run the trunk fmt again

It's failing on my laptop when trying to format my png image, due to execve failed: Bad CPU type in executable. I'm tinkering to see if I can make it work, but I feel like I might be going about this the wrong way. Is there a way to run the formatter here within the PR? Or should I have had some pre-commit hooks already set up for this?

@glaracuente
Copy link
Copy Markdown
Contributor Author

I had a 1-1 with @oycyc and he explained that we use trunk fmt across many repos and that we run it locally. One of the tools that trunk uses is oxipng when optimizing png images. For some reason, it was trying to use an x86_64 executable of this tool and I had to force it to use the proper arm64 version for my laptop to finally get it working.

@glaracuente glaracuente merged commit 01e6a1c into master Mar 31, 2026
8 checks passed
@glaracuente glaracuente deleted the feature/INT-332/add-gerry-to-website branch March 31, 2026 19:15
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