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

[TASK] Sort local extension list for editing #763

Conversation

DavidBruchmann
Copy link
Contributor

fix #760

@DavidBruchmann
Copy link
Contributor Author

to clarify my rewording in the comments:
We'll have to deal with remote extensions from TER too, so the comments have to be clear about the kind of extensions to avoid any confusion and misunderstanding.

@PKuhlmay
Copy link
Collaborator

PKuhlmay commented Apr 4, 2024

I added the patch via cherry pick and close this PR. Reason is: Something strange happened, there are some commits inside this PR, which should be inside the target branch already.

Thanks for sorting the extensions. It worked for me.

@DavidBruchmann
Copy link
Contributor Author

Yeah thanks for picking, I knew I shouldn't have clicked the button for merging something 😆

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.

2 participants