Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: add customGroups.fallbackSort option #474

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

hugop95
Copy link
Contributor

@hugop95 hugop95 commented Feb 19, 2025

Follow-up of #457 and #469.

Description

This PR does the following:

  • Adds a customGroups.fallbackSort option to rules that support the array-based customGroups API.
  • Adds the fallbackSort option to settings.
  • Improves documentation readability.
  • Refactors compare, sort-nodes and sort-nodes-by-groups in order to prepare for the next PR that will allow us to finally let sort-object-types and sort-interfaces handle customGroups.fallbackSort.sortBy.
  • Adds missing tests.

What is the purpose of this pull request?

  • New Feature

@hugop95 hugop95 marked this pull request as ready for review February 19, 2025 13:04
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant