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.

Sign up for GitHub

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

Jump to bottom

fmt_test: resolve absolute path issue of fmt testing #2562

Merged

ry merged 1 commit into denoland:master from kt3k:feature/resolve-fmt-test-todo

Jun 22, 2019

Member

kt3k commented Jun 22, 2019

This PR resolves a TODO comment in //tools/fmt_test.py

deno fmt used to has a bug about handling of absolute paths. Now it was fixed at denoland/std#438, and we can use absolute paths in arguments.


          fmt_test: resolve old absolute path issue

8cbb8de

kt3k changed the title ~~fmt_test: resolve old absolute path issue~~ fmt_test: resolve absolute path issue

kt3k changed the title ~~fmt_test: resolve absolute path issue~~ fmt_test: resolve absolute path issue of fmt testing

ry approved these changes

View reviewed changes

Member

ry left a comment

LGTM

ry merged commit 201ddd2 into denoland:master

kt3k deleted the feature/resolve-fmt-test-todo branch

June 22, 2019 16:17

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet