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

Port https://github.com/nats-io/nats-server/pull/3887 to dev #3912

Merged
merged 1 commit into from Feb 27, 2023

Conversation

jnmoyne
Copy link
Contributor

@jnmoyne jnmoyne commented Feb 27, 2023

  • Tests added
  • Branch rebased on top of current dev (git pull --rebase origin main)
  • Changes squashed to a single commit (described here)
  • Build is green in Travis CI
  • You have certified that the contribution is your original work and that you license the work to the project under the Apache 2 license

Applies #3887 to dev (since I noticed the merge of main to dev of that PR effectively made it disappear (because stream transform changes included extracting transforms from accounts.go)

… (since the merge of main to dev of that PR effectively made it disappear (because stream transform changes included extracting transforms from accounts.go)
@jnmoyne jnmoyne requested a review from a team as a code owner February 27, 2023 01:19
@derekcollison
Copy link
Member

Make sure tests green first.

Copy link
Member

@derekcollison derekcollison left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@derekcollison derekcollison merged commit c70290a into dev Feb 27, 2023
@derekcollison derekcollison deleted the jnm/dev_26Feb23 branch February 27, 2023 04:39
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.

None yet

2 participants