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

Clean up root directory #3247

Merged
merged 15 commits into from
Nov 1, 2019
Merged

Clean up root directory #3247

merged 15 commits into from
Nov 1, 2019

Conversation

ry
Copy link
Member

@ry ry commented Oct 31, 2019

No description provided.

@ry ry requested a review from piscisaureus October 31, 2019 15:45
@hayd
Copy link
Contributor

hayd commented Oct 31, 2019

Another thing you could do is to roll .eslintignore .eslintrc .prettierrc etc into package.json

@ry
Copy link
Member Author

ry commented Oct 31, 2019

@hayd That would be amazing - I hate all these random configuration files.

Copy link
Member

@piscisaureus piscisaureus left a comment

Choose a reason for hiding this comment

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

👏

@piscisaureus piscisaureus merged commit af61dbe into denoland:master Nov 1, 2019
hayd added a commit to hayd/deno that referenced this pull request Nov 5, 2019
Keeping a empty node_modules was required, because the test relied on the node_modules symlink to point to a valid target. This is no longer the case since denoland#3247, so we can remove this workaround too.

Co-Authored-By: Christian Moritz <[email protected]>
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.

3 participants