Skip to content

feat: e2e rule to have assignee comment on test.fixme#31

Open
harshal015 wants to merge 1 commit into
masterfrom
require-test-assignee-comment
Open

feat: e2e rule to have assignee comment on test.fixme#31
harshal015 wants to merge 1 commit into
masterfrom
require-test-assignee-comment

Conversation

@harshal015

Copy link
Copy Markdown
Contributor

Description

app.clickup.com
Brief description of the changes made in this PR.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Performance improvement
  • Refactoring (no functional changes)

Testing

  • Tests added/updated for new functionality
  • All existing tests pass
  • Manual testing completed
  • Edge cases tested

Checklist

  • Code follows the project's style guidelines
  • Self-review of code completed
  • Documentation updated (if applicable)
  • No breaking changes (or breaking changes are documented)
  • Changes are backward compatible
  • Linting passes (npm run lint)
  • Tests pass (npm test)
  • Type checking passes (npm run type-check)

Related Issues

Closes #(issue number)

Screenshots (if applicable)

Add screenshots to help explain your changes.

Additional Notes

Any additional information that reviewers should know.

@harshal015 harshal015 requested a review from rvab as a code owner May 25, 2026 07:08
schema: [],
messages: {
missingAssignee:
'test.fixme/test.fail must include an explicit assignee comment immediately above the call, e.g. "// @assignee @username".',

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

  • Should the comment include @assignee or just assignee?
  • Will it work for multiline comment?

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Plus 1

@harshal015 harshal015 requested a review from OmkarJ13 May 25, 2026 10:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants