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.

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

refactor: sort imports #105

Merged
merged 1 commit into from
Jan 23, 2024
Merged

Conversation

jaypaik
Copy link
Collaborator

@jaypaik jaypaik commented Jan 23, 2024

Sorting all the imports by path using a VS Code plugin. Following this general structure, with a newline separator in between each:

  1. forge-std imports, if this is a test file.
  2. External dependencies
  3. Source files

@jaypaik jaypaik force-pushed the 01-22-refactor_move_test_files_to_appropriate_folders branch from 2c7cd00 to ba4e918 Compare January 23, 2024 04:38
@jaypaik jaypaik force-pushed the 01-22-refactor_sort_imports branch from c4088b4 to c52dc9c Compare January 23, 2024 04:38
@jaypaik jaypaik force-pushed the 01-22-refactor_move_test_files_to_appropriate_folders branch from ba4e918 to 936c6ca Compare January 23, 2024 05:16
Base automatically changed from 01-22-refactor_move_test_files_to_appropriate_folders to audit-2023-11-20 January 23, 2024 05:16
@jaypaik jaypaik force-pushed the 01-22-refactor_sort_imports branch from c52dc9c to ae356aa Compare January 23, 2024 05:17
@jaypaik jaypaik force-pushed the 01-22-refactor_sort_imports branch from ae356aa to 21ca57c Compare January 23, 2024 05:20
@jaypaik jaypaik merged commit 52a5577 into audit-2023-11-20 Jan 23, 2024
3 checks passed
@jaypaik jaypaik deleted the 01-22-refactor_sort_imports branch January 23, 2024 05:32
jaypaik added a commit that referenced this pull request Jan 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants