Skip to content

Comments

Release/26.1.5#277

Open
skyflow-bharti wants to merge 2 commits intomainfrom
release/26.1.5
Open

Release/26.1.5#277
skyflow-bharti wants to merge 2 commits intomainfrom
release/26.1.5

Conversation

@skyflow-bharti
Copy link
Collaborator

Start with a concise summary of the PR. The first three sections are required. The questions present in each section is there to help you guide you what to add. They are meant to be overwritten by your comments.

Why

  • Why are you making the change?
  • What is the underlying issue that you are trying to case, in case of fix?
  • Why is it needed by the feature you are working on?
  • What is the intent behind making the change?

Goal

  • What is the intended outcome?
  • What part of the feature should start working?
  • What are the non-goals or will be covered in future PR?

Testing

  • How was the code tested?
  • If you haven't written unit tests, why?
  • What more testing is needed? Do you intend to manually test it after deployment?
  • Do you have any concerns if this changed is released to prod?

Tech debt

  • Is the PR adding to tech debt in any way?
  • Are you addressing some Tech debt in this PR?
  • If both the above are false, feel free to remove this section.

skyflow-himanshu and others added 2 commits February 5, 2026 10:17
#272)

* SK-2519: extract hard coded strings to constants and update linting rules

* aadarsh-st/SK-2521-Enhance content-type handling and add validation tests for request bodies
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.

3 participants