-
Notifications
You must be signed in to change notification settings - Fork 352
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: Add Rewrite Header Annotations #2285
base: master
Are you sure you want to change the base?
Conversation
c6eb07d
to
3eea3f4
Compare
I am unable to reproduce the failure of the e2e test locally. Does somebody has an idea why the test fails (maybe flaky)? |
@pottekkat Can I do anything that the PR will be reviewed? |
@mpoqq Can you fix one failing lint check? |
@Revolyssup I would like to do that, but the lint errors are the same as in the main branch. They should probably be fixed in a separate PR. |
Got it. |
I will leave a review by the evening. |
@Revolyssup Friendly Reminder 😛 |
This pull request has been marked as stale due to 60 days of inactivity. It will be closed in 30 days if no further activity occurs. If you think that's incorrect or this pull request should instead be reviewed, please simply write any comment. Even if closed, you can still revive the PR at any time or discuss it on the [email protected] list. Thank you for your contributions. |
Type of change:
What this PR does / why we need it:
Pre-submission checklist:
Implements #2284
Adds Rewrite Add/Set/Remove Header annotations.