Skip to content
This repository has been archived by the owner on Sep 30, 2024. It is now read-only.

Sort use alphabetically. #134

Merged
merged 1 commit into from
Jun 22, 2023
Merged

Sort use alphabetically. #134

merged 1 commit into from
Jun 22, 2023

Conversation

edyuenyw
Copy link
Contributor

@edyuenyw edyuenyw commented Jun 22, 2023 •
edited
Loading

Sort use alphabetically.

@edyuenyw edyuenyw self-assigned this Jun 22, 2023
krakerag
krakerag approved these changes Jun 22, 2023
View reviewed changes
Copy link

@krakerag krakerag left a comment

Choose a reason for hiding this comment

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

LGTM

I'm sure there are better things we can be linting for rather than this... but if the linter says so, gotta be done right :)

@edyuenyw edyuenyw merged commit 390fcb3 into develop Jun 22, 2023
@edyuenyw edyuenyw deleted the slevomat-coding-standards-sort-use-alphabetically branch June 22, 2023 04:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@krakerag krakerag krakerag approved these changes

@anthony-malkoun anthony-malkoun Awaiting requested review from anthony-malkoun

@vincent-gao vincent-gao Awaiting requested review from vincent-gao

@mayurngondhkar mayurngondhkar Awaiting requested review from mayurngondhkar

@MdNadimHossain MdNadimHossain Awaiting requested review from MdNadimHossain

@yeniatencio yeniatencio Awaiting requested review from yeniatencio

Assignees

@edyuenyw edyuenyw

Labels
Needs review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@edyuenyw @krakerag