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

fix: load all eagerly loaded files and folders #14

Merged
merged 1 commit into from
Oct 15, 2024

Conversation

renanccastro
Copy link
Contributor

@renanccastro renanccastro commented Oct 15, 2024

Load all eagerly loaded files and folders

Problem

Previously, the linter was only checking files within the server folder, which could miss eagerly loaded files and folders outside of this directory. This led to incomplete linting coverage for Meteor projects not using main modules.

Solution

This PR modifies the Walker class to start from the root app directory instead of specifically the server folder. This change ensures that all eagerly loaded files and folders are included in the linting process, regardless of their location in the project structure.

Changes

  • Updated lib/util/walker.js to start walking from the app root directory
  • Removed the check for the existence of the server folder
  • Updated CHANGELOG.md to reflect the bug fix in versions 1.5.1 and 1.5.2
  • Bumped package version to 1.5.2 in package.json

@renanccastro renanccastro merged commit 6c83171 into main Oct 15, 2024
2 checks passed
@renanccastro renanccastro deleted the fix/eagerly-loaded branch October 15, 2024 17:18
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