Skip to content

Conversation

tiwillia
Copy link
Contributor

Provides comprehensive guidance for Claude Code including:

  • Development commands for testing and linting
  • Architecture overview of OCM common library
  • Package descriptions and logging best practices

🤖 Generated with Claude Code

@openshift-ci openshift-ci bot requested review from macgregor and markturansky July 31, 2025 01:17
Copy link

openshift-ci bot commented Jul 31, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: tiwillia
Once this PR has been reviewed and has the lgtm label, please assign macgregor for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@rafabene
Copy link
Contributor

rafabene commented Aug 4, 2025

Please add pkg/test and pkg/middleware/deprecation.go

tiwillia and others added 2 commits August 4, 2025 16:48
Provides comprehensive guidance for Claude Code including:
- Development commands for testing and linting
- Architecture overview of OCM common library
- Package descriptions and logging best practices

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
- Add comprehensive documentation for pkg/test OCM API testing framework
- Include details on concurrent execution, label-based filtering, and continuous testing
- Add pkg/middleware/deprecation.go to middleware section
- Document authentication support and usage patterns

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
@tiwillia
Copy link
Contributor Author

tiwillia commented Aug 4, 2025

@rafabene updated. PTAL.

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