Skip to content

[Messenger] Use same buses names #11974

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

Merged
merged 1 commit into from
Aug 2, 2019
Merged

[Messenger] Use same buses names #11974

merged 1 commit into from
Aug 2, 2019

Conversation

Lctrs
Copy link
Contributor

@Lctrs Lctrs commented Jul 17, 2019

No description provided.

@javiereguiluz javiereguiluz added this to the 4.3 milestone Jul 18, 2019
@OskarStark
Copy link
Contributor

Could you please squash your commits into one? We cannot squash merge it with our tool if there is a merge commit.

Thank you!

@javiereguiluz
Copy link
Member

@OskarStark the branch name patch-1 is usually associated with quick edits via gitHub's web interface ... so "squashing" is not possible in a simple way.

@OskarStark
Copy link
Contributor

I know but how does the merge commit goes in then? 🧐

@Lctrs
Copy link
Contributor Author

Lctrs commented Jul 25, 2019

The merge commit is from the built in merge conflict tool of GitHub.

I'm currently on holidays. Will do it when I come back home on august, 1st.

@Lctrs
Copy link
Contributor Author

Lctrs commented Aug 2, 2019

@OskarStark Should be good now.

@OskarStark
Copy link
Contributor

Thank you @Lctrs.

@OskarStark OskarStark merged commit b33a14a into symfony:4.3 Aug 2, 2019
OskarStark added a commit that referenced this pull request Aug 2, 2019
This PR was merged into the 4.3 branch.

Discussion
----------

[Messenger] Use same buses names

Commits
-------

b33a14a [Messenger] Use same buses names
@Lctrs Lctrs deleted the patch-1 branch August 2, 2019 10:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants