Skip to content

Conversation

@ViniTou
Copy link
Contributor

@ViniTou ViniTou commented Sep 3, 2021

Sidenote:

Should we merge those via merge&rebase or merge commit?

@ViniTou ViniTou requested a review from a team September 3, 2021 09:30
@alongosz alongosz added the Fast-forward merge PR should be merged in a fast-forward way label Sep 7, 2021
@alongosz alongosz changed the title Aligned code [Fast forward] Aligned code Sep 7, 2021
@alongosz
Copy link
Member

alongosz commented Sep 7, 2021

@ViniTou merge commit should never be used and it should be disabled in all Ibexa & eZ Systems repositories. Unfortunately I don't have access to settings in org:ibexa to so.
Any PRs containing merges of other PRs should be merged in a fast forward way (which means rebase and merge)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Fast-forward merge PR should be merged in a fast-forward way

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants