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

Add RepositoryFilesystem #21

Merged
merged 5 commits into from
Nov 9, 2023
Merged

Add RepositoryFilesystem #21

merged 5 commits into from
Nov 9, 2023

Conversation

mcmire
Copy link
Collaborator

@mcmire mcmire commented Oct 26, 2023

When linting a project, we need to constantly read the files within either the module template or the project. We want this tool to run as fast as possible, and to do this, we can cache the metadata and content of each file that we read so that we don't have to do it again. This class assists with that.

Related to #5.


See here for context on how a RepositoryFilesystem will eventually be used:

When linting a project, we need to constantly read the files within
either the module template or the project. We want this tool to run as
fast as possible, and to do this, we can cache the metadata and content
of each file that we read so that we don't have to do it again. This
class assists with that.
@mcmire mcmire requested a review from a team as a code owner October 26, 2023 23:16
@socket-security
Copy link

socket-security bot commented Oct 26, 2023

New dependencies detected. Learn more about Socket for GitHub ↗︎

Packages Version New capabilities Transitives Size Publisher
@metamask/utils 8.2.0 filesystem +17 4.96 MB metamaskbot

@@ -9,6 +9,9 @@
"noEmit": true,
"noErrorTruncation": true,
"noUncheckedIndexedAccess": true,
"paths": {
"@metamask/utils/node": ["./node_modules/@metamask/utils/dist/types/node"]
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

The reason we're setting paths is explained here: #20 (comment)

Gudahtt
Gudahtt previously approved these changes Nov 6, 2023
Copy link
Member

@Gudahtt Gudahtt left a comment

Choose a reason for hiding this comment

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

LGTM!

Co-authored-by: Mark Stacey <markjstacey@gmail.com>
Gudahtt
Gudahtt previously approved these changes Nov 7, 2023
@mcmire
Copy link
Collaborator Author

mcmire commented Nov 7, 2023

^ Addressed merge conflicts.

Copy link
Member

@Gudahtt Gudahtt left a comment

Choose a reason for hiding this comment

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

LGTM!

@mcmire mcmire merged commit dd455ba into main Nov 9, 2023
22 checks passed
@mcmire mcmire deleted the add-repository-filesystem branch November 9, 2023 19:03
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