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

Prettier: support for specified files and glob mode #438

Merged
merged 6 commits into from
May 25, 2019

Conversation

axetroy
Copy link
Contributor

@axetroy axetroy commented May 23, 2019

close #424

Thanks to @dshubhadeep for #433

support specified files mix with glob

> deno run prettier/main.ts script1.ts script2.ts **/**/test.ts

This allows to directly specify a file to be formatted.

Before

> deno run prettier/main.ts test.ts
Formatting test.ts
Formatting fs/test.ts
Formatting fs/copy_test.ts
Formatting fs/empty_test.ts
...

After

only format the test.ts

> deno run prettier/main.ts test.ts
Formatting test.ts

/cc @kt3k

prettier/main.ts Outdated Show resolved Hide resolved
prettier/main.ts Outdated Show resolved Hide resolved
@kt3k
Copy link
Member

kt3k commented May 25, 2019

otherwise looks good to me.

Copy link
Member

@ry ry left a comment

Choose a reason for hiding this comment

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

LGTM

@ry ry merged commit 1083db1 into denoland:master May 25, 2019
@axetroy axetroy deleted the prettier branch May 25, 2019 14:18
ry pushed a commit to ry/deno that referenced this pull request Oct 9, 2019
inverted-capital pushed a commit to dreamcatcher-tech/napps that referenced this pull request Oct 24, 2024
This change adds the following REST API endpoints:
- `GET /api/items`
- `GET /api/items/[id]`
- `GET /api/items/[id]/comments`

Documentation will come in a follow-up PR.

Prerequisite for denoland#438 or denoland#429
Towards denoland#439
inverted-capital pushed a commit to dreamcatcher-tech/napps that referenced this pull request Oct 24, 2024
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.

[discussion] Do not use glob for prettier by default
3 participants