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

chore: lint server_test.ts #19175

Merged
merged 1 commit into from
May 18, 2023
Merged

Conversation

bartlomieju
Copy link
Member

Somehow this file had // deno-lint-ignore-file directive at the top.

This commit removes it and fixes all the linter violations.

@bartlomieju bartlomieju requested review from mmastrac and dsherret May 18, 2023 01:20
@bartlomieju bartlomieju enabled auto-merge (squash) May 18, 2023 01:20
@mmastrac
Copy link
Contributor

Tests failing :(

@bartlomieju
Copy link
Member Author

Tests failing :(

Shouldn't anymore

Copy link
Contributor

@mmastrac mmastrac left a comment

Choose a reason for hiding this comment

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

LGTM assuming tests pass with last commit

@bartlomieju bartlomieju merged commit 99c3028 into denoland:main May 18, 2023
@bartlomieju bartlomieju deleted the lint_serve_test branch May 18, 2023 10:57
levex pushed a commit that referenced this pull request May 18, 2023
Somehow this file had `// deno-lint-ignore-file` directive at the top.

This commit removes it and fixes all the linter violations.
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