Skip to content

fix: remove broken greet-new-contributors workflow#77

Merged
michen00 merged 2 commits into
mainfrom
hotfix/remove-greet-contributors-workflow
May 20, 2026
Merged

fix: remove broken greet-new-contributors workflow#77
michen00 merged 2 commits into
mainfrom
hotfix/remove-greet-contributors-workflow

Conversation

@michen00
Copy link
Copy Markdown
Owner

@michen00 michen00 commented May 20, 2026

Summary

Test plan

  • Pre-commit passed on commit
  • Merge and confirm no greet new contributors workflow runs on new issues/PRs

See also

The actions/first-interaction greeting still referenced the upstream
invisible-squiggles template and posted off-brand bot comments on new
issues and PRs.
@qodo-code-review
Copy link
Copy Markdown

Review Summary by Qodo

Remove broken greet-new-contributors workflow

🐞 Bug fix

Grey Divider

Walkthroughs

Description
• Removes broken greet-new-contributors workflow file
• Deletes outdated actions/first-interaction@v3 configuration
• Removes yamllint exception for deleted workflow
• Eliminates off-brand bot comments on issues/PRs
Diagram
flowchart LR
  A["greet-new-contributors.yml<br/>with stale config"] -- "delete" --> B["Workflow removed"]
  C["yamllint config<br/>with exception"] -- "update" --> D["Exception removed"]
  B --> E["No bot comments<br/>on new issues/PRs"]
  D --> E
Loading

Grey Divider

File Changes

1. .github/workflows/greet-new-contributors.yml 🐞 Bug fix +0/-36

Delete greet-new-contributors workflow

• Deleted entire workflow file (36 lines)
• Removed actions/first-interaction@v3 action with stale upstream template references
• Eliminated off-brand bot greeting messages for issues and pull requests

.github/workflows/greet-new-contributors.yml


2. .yamllint ⚙️ Configuration changes +0/-1

Remove yamllint exception for deleted workflow

• Removed yamllint line-length exception for deleted greet-new-contributors.yml file
• Kept exception for pre-commit-autoupdate.yml workflow

.yamllint


Grey Divider

Qodo Logo

@qodo-code-review
Copy link
Copy Markdown

qodo-code-review Bot commented May 20, 2026

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Qodo Logo

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the .yamllint configuration by removing the .github/workflows/greet-new-contributors.yml file from the line-length ignore list, ensuring it now adheres to the standard 100-character limit. No review comments were provided for these changes, and I have no feedback to provide.

@michen00 michen00 self-assigned this May 20, 2026
@michen00 michen00 merged commit 1526cdb into main May 20, 2026
10 checks passed
@michen00 michen00 deleted the hotfix/remove-greet-contributors-workflow branch May 20, 2026 02:01
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.

1 participant