Skip to content
This repository has been archived by the owner on Oct 6, 2023. It is now read-only.

Tests, linting, & formatting #69

Merged
merged 3 commits into from
May 26, 2023
Merged

Tests, linting, & formatting #69

merged 3 commits into from
May 26, 2023

Conversation

SovereignAndrey
Copy link
Contributor

Closes #38

Explanation of the solution

  • Add prettier package, a yarn format command
  • runs the formatter across all the files js/ts/sol

Instructions on making this work

None

@SovereignAndrey SovereignAndrey added enhancement New feature or request good first issue Good for newcomers labels May 26, 2023
@SovereignAndrey SovereignAndrey requested a review from 0xNeshi May 26, 2023 04:41
@SovereignAndrey SovereignAndrey self-assigned this May 26, 2023
0xNeshi
0xNeshi approved these changes May 26, 2023
View reviewed changes
Copy link
Contributor

@0xNeshi 0xNeshi left a comment

Choose a reason for hiding this comment

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

The code is so much more pleasant to look at now that every file uses the same format. Nice!

@SovereignAndrey SovereignAndrey merged commit 597b4d6 into master May 26, 2023
@SovereignAndrey SovereignAndrey deleted the tests-linting-formatting branch May 26, 2023 06:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@0xNeshi 0xNeshi 0xNeshi approved these changes

Assignees

@SovereignAndrey SovereignAndrey

Labels
enhancement New feature or request good first issue Good for newcomers
Projects
No open projects
Angel Protocol dApp
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add common styles/code formats using prettier
2 participants
@SovereignAndrey @0xNeshi