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

feat(customRule): allow reordering rules #229

Closed
wants to merge 1 commit into from

Conversation

Mara-Li
Copy link
Contributor

@Mara-Li Mara-Li commented Sep 11, 2023

See #71 and #226

Probably not the best way to add this

Note

If two rules apply on the same files, the plugin doesn't know which one to use and blink between the icons. Dunno why, to be honest…

@FlorianWoelki
Copy link
Owner

This functionality will probably work 90% of the time. However, I would like to have the functionality that the order is actually saved in the rules data and use this order to define the order and priority of the custom rules. That's how we can verify that it works 100% of the time.

@FlorianWoelki FlorianWoelki self-requested a review September 12, 2023 06:46
@Mara-Li
Copy link
Contributor Author

Mara-Li commented Sep 12, 2023

Yeah, that's what I thinked in second but dunno how to implement this as it need to update old rules too!

@FlorianWoelki
Copy link
Owner

Yeah, that's what I thinked in second but dunno how to implement this as it need to update old rules too!

I'll do this in the future and contribute to this PR.

@FlorianWoelki
Copy link
Owner

I made some tweaks, feel free to checkout #237

@FlorianWoelki
Copy link
Owner

Will close because of #237

@Mara-Li Mara-Li deleted the reordering-rules branch October 1, 2023 09:47
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