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.

Sign up for GitHub

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

Jump to bottom

refactor(*): combine multiple checks #886

Merged
merged 3 commits into from
Apr 29, 2023
Merged

refactor(*): combine multiple checks #886

merged 3 commits into from
Apr 29, 2023

Conversation

Elsie19
Copy link
Member

@Elsie19 Elsie19 commented Apr 29, 2023

Purpose

Because we can and it looks cleaner.

Checklist

  • I confirm that I have read the contributing guidelines, and this pull request is abiding by all the clauses stated in the guideline.

@Elsie19
refactor(*): combine multiple checks
af5606d 
Hey Wizard, are you happy I'm using refactor instead of ref :)
@Elsie19 Elsie19 added enhancement New feature or request refactor Refactors code release notes Something that is or should be mentioned in the release notes cleanup Cleans up code bash For the bash version of Pacstall labels Apr 29, 2023
@Elsie19 Elsie19 merged commit 1c91fba into pacstall:develop Apr 29, 2023
2 checks passed
@Elsie19 Elsie19 deleted the more-conversion branch April 29, 2023 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wizard-28 wizard-28 wizard-28 approved these changes

@D-Brox D-Brox Awaiting requested review from D-Brox D-Brox is a code owner

Assignees
No one assigned
Labels
bash For the bash version of Pacstall cleanup Cleans up code enhancement New feature or request refactor Refactors code release notes Something that is or should be mentioned in the release notes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@Elsie19 @wizard-28