-
Notifications
You must be signed in to change notification settings - Fork 320
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
fix: [Bug - 4658] Updated stop sequence to empty string in Bicep #1070
Conversation
The title of your PR is used in the Release Notes - are you able to update it so that it is a meaningful sentence (I think you have this in the description) |
@ross-p-smith Done |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please could you make sure that main.json
also gets updated
az bicep build -f infra/main.bicep
Yes @cecheta - Its updated. Please review and approve once. |
🎉 This PR is included in version 1.7.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Purpose
Changed the default value from "\n" to empty in Bicep to get the complete responses
Does this introduce a breaking change?
How to Test
Run the app and check whether we are getting complete response for every question