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

Document MQM support #122

Closed
wants to merge 31 commits into from
Closed

Document MQM support #122

wants to merge 31 commits into from

Conversation

zouharvi
Copy link
Contributor

@zouharvi zouharvi commented Feb 8, 2024

This is a draft pull-request for @snukky and @kocmitom. Ultimately the aim is to support both full MQM and a slimmed-down version of it.

There's a lot of commits so if this gets merged, then please squash-merge.

@snukky snukky marked this pull request as ready for review February 9, 2024 16:12
@snukky snukky self-requested a review February 9, 2024 16:12
@snukky
Copy link
Contributor

snukky commented Feb 9, 2024

Thanks! The newest branch that we use is release (sorry, I should have make it clear from the beginning). Can you rebase your branch with it and (temporarily) open against it?

@snukky
Copy link
Contributor

snukky commented Feb 9, 2024

It would also be great to cherry-pick the changes related to switching to Django 5 and open a separate PR, so that we can test that independently.

@zouharvi
Copy link
Contributor Author

zouharvi commented Feb 9, 2024

Ah, I didn't know that release was the live branch. I opened a different PR for the django-5 two weeks ago but I messed-up the rebase.

I'll open two new PRs then.

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.

2 participants