Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: APIRule v2 introduction #1663

Open
wants to merge 16 commits into
base: main
Choose a base branch
from

Conversation

kubicar
Copy link
Contributor

@kubicar kubicar commented Feb 3, 2025

Description
Changes proposed in this pull request:
This pull request introduces a new version (v2) for the APIRule, conversion functions, and related e2e tests which stays same as for v2alpha1

Pre-Merge Checklist

  • As a PR reviewer, verify code coverage and evaluate if it is acceptable.

Related issues

@kyma-bot kyma-bot added do-not-merge/work-in-progress cla: yes Indicates the PR's author has signed the CLA. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Feb 3, 2025
@kubicar kubicar changed the title feat: introduce v2 APIRule API with conversion feat: APIRule v2 introduction Feb 5, 2025
@kubicar kubicar marked this pull request as ready for review February 5, 2025 10:55
@kubicar kubicar requested a review from a team as a code owner February 5, 2025 10:55
@kubicar kubicar mentioned this pull request Feb 5, 2025
12 tasks
@kubicar kubicar requested a review from a team as a code owner February 10, 2025 10:04
nataliasitko
nataliasitko previously approved these changes Feb 12, 2025
@kyma-bot kyma-bot added the lgtm Looks good to me! label Feb 12, 2025
@kubicar
Copy link
Contributor Author

kubicar commented Feb 12, 2025

be sure that test ext_auth for v2 will be added after removal of v2alpha1

@kyma-bot kyma-bot removed the lgtm Looks good to me! label Feb 12, 2025
@kyma-bot
Copy link
Contributor

New changes are detected. LGTM label has been removed.

@Ressetkk
Copy link
Contributor

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Indicates the PR's author has signed the CLA. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants