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

fix new lines in snippets #4832

Merged
merged 5 commits into from
Dec 21, 2023
Merged

Conversation

j1m-ryan
Copy link

@j1m-ryan j1m-ryan commented Dec 21, 2023

Proposed changes

#4824

Fixes server snippet new lines issue in vservers and transport servers by including a new line in the tmpl files

Checklist

Before creating a PR, run through this checklist and mark each as complete.

  • I have read the CONTRIBUTING doc
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked that all unit tests pass after adding my changes
  • I have updated necessary documentation
  • I have rebased my branch onto main
  • I will ensure my PR is targeting the main branch and pulling from my branch from my own fork

@j1m-ryan j1m-ryan requested a review from a team as a code owner December 21, 2023 12:16
Copy link

netlify bot commented Dec 21, 2023

👷 Deploy request for nginx-kubernetes-ingress pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 32b3a52

@github-actions github-actions bot added the bug An issue reporting a potential bug label Dec 21, 2023
@j1m-ryan j1m-ryan marked this pull request as draft December 21, 2023 12:16
Copy link

codecov bot commented Dec 21, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (95f8da6) 52.13% compared to head (32b3a52) 52.10%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4832      +/-   ##
==========================================
- Coverage   52.13%   52.10%   -0.03%     
==========================================
  Files          60       60              
  Lines       17368    17368              
==========================================
- Hits         9054     9050       -4     
- Misses       8000     8002       +2     
- Partials      314      316       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@github-actions github-actions bot added the tests Pull requests that update tests label Dec 21, 2023
@j1m-ryan j1m-ryan marked this pull request as ready for review December 21, 2023 13:09
@vepatel vepatel linked an issue Dec 21, 2023 that may be closed by this pull request
@j1m-ryan j1m-ryan enabled auto-merge (squash) December 21, 2023 14:31
@j1m-ryan j1m-ryan merged commit f63f0db into nginx:main Dec 21, 2023
17 checks passed
@centromere
Copy link
Contributor

Thank you for the fast response!

pdabelf5 pushed a commit that referenced this pull request Feb 9, 2024
* fix new lines in snippets

* add test for server snippet new lines

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
pdabelf5 added a commit that referenced this pull request Feb 9, 2024
* fix new lines in snippets (#4832)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug An issue reporting a potential bug tests Pull requests that update tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Newlines are not preserved in server snippets
5 participants