Dark Mode

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

TestRule.test_dump_rule_file: sort the rule file lists.#3582

Merged
AyanSinhaMahapatra merged 1 commit intoaboutcode-org:developfrom
licquia:test_dump_rule_file_fix
Nov 15, 2023
Merged

TestRule.test_dump_rule_file: sort the rule file lists.#3582
AyanSinhaMahapatra merged 1 commit intoaboutcode-org:developfrom
licquia:test_dump_rule_file_fix

Conversation

Copy link
Contributor

licquia commented Nov 13, 2023 *
edited by AyanSinhaMahapatra
Loading

Fixes #3581.

Tasks

  • Reviewed contribution guidelines
  • PR is descriptively titled and links the original issue above
  • Tests pass -- look for a green checkbox a few minutes after opening your PR
    Run tests locally to check for errors.
  • Commits are in uniquely-named feature branch and has no merge conflicts
  • Looked for possible updates in documentation and added updates if applicable
  • Updated CHANGELOG.rst

pombredanne approved these changes Nov 13, 2023
Copy link
Member

pombredanne left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thank you ++

Copy link
Member

AyanSinhaMahapatra commented Nov 15, 2023

Thanks ++ @licquia , merging!

AyanSinhaMahapatra merged commit 7b838ab into aboutcode-org:develop Nov 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

pombredanne pombredanne approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

test_dump_rule_file depends on OS filename sorting

3 participants