Skip to content

Conversation

@tomkingkong
Copy link
Contributor

Proposed changes

Add Tom King to the team docs.

@claude
Copy link
Contributor

claude bot commented Oct 28, 2025

Review Findings

Thank you for adding Tom King's profile! I've reviewed the changes and found one issue that needs to be addressed:

Issues Found

Missing newline at end of file (data/team/team/tom-king.toml:10)

The file must end with a newline character. According to AGENTS.md, "Markdown: Must always end with a newline." This applies to all text files in the repository.

Suggested Fix

[social]
github = "tomkingkong"
bluesky = "@tomking.design"
linkedin = "tom-king-12a102b"

(Add a blank line after the last line)

Format Observations

The file structure looks good overall and follows the existing team profile format. I noticed:

  • ✅ All required fields are present (id, name, status, title, weight)
  • ✅ Status is correctly set to "active"
  • ✅ Social media fields are properly formatted
  • ✅ Field ordering is consistent with other profiles

Note on Social Fields

The file uses "bluesky" as a social field. This appears to be a new platform addition that other profiles don't have yet. Please verify this is supported by the Hugo templates that render these team profiles.


Once the newline issue is fixed, this PR will be ready to merge. Let me know if you'd like me to make the fix, or mention @claude if you need additional reviews or assistance.

@pulumi-bot
Copy link
Collaborator

@pulumi-bot
Copy link
Collaborator

@pulumi-bot
Copy link
Collaborator

Copy link
Contributor

@CamSoper CamSoper left a comment

Choose a reason for hiding this comment

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

Looks fine, let's :shipit:

@tomkingkong tomkingkong enabled auto-merge (squash) October 28, 2025 17:16
@pulumi-bot
Copy link
Collaborator

@tomkingkong tomkingkong merged commit bbe4d34 into master Oct 28, 2025
8 checks passed
@tomkingkong tomkingkong deleted the tk/tom-profile branch October 28, 2025 17:25
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.

4 participants