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

Update README.md #50

Closed
wants to merge 1 commit into from
Closed

Update README.md #50

wants to merge 1 commit into from

Conversation

bn-l
Copy link

@bn-l bn-l commented Mar 7, 2024

Improved syntax and clarity for someone who hasn't parsed the entire repo.

Improved syntax and clarity for someone who hasn't parsed the entire repo.
@@ -35,9 +35,9 @@ Both keys and values of the `rewritePaths` object must end with a `/`. Paths are

You can enable compilation via the `compile` property. If `false`, AVA will assume you have already compiled your project. If set to `'tsc'`, AVA will run the TypeScript compiler before running your tests. This can be inefficient when using AVA in watch mode.

Output files are expected to have the `.js` extension.
Output files are expected to have the `.js` / `.mjs` / `.cjs` extension.
Copy link
Member

Choose a reason for hiding this comment

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

👍🏻


AVA searches your entire project for `*.js`, `*.cjs`, `*.mjs`, `*.ts`, `*.cts` and `*.mts` files (or other extensions you've configured). It will ignore such files found in the `rewritePaths` targets (e.g. `build/`). If you use more specific paths, for instance `build/main/`, you may need to change AVA's `files` configuration to ignore other directories.
AVA will now treat `*.ts`, `*.cts` and `*.mts` files as tests (but really it's running the compiled javascript). It will ignore found in the `rewritePaths` targets (e.g. `build/`). If you use more specific paths, for instance `build/main/`, you may need to change AVA's `files` [configuration](https://github.com/avajs/ava/blob/main/docs/06-configuration.md#configuration) to ignore other directories.
Copy link
Member

Choose a reason for hiding this comment

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

I don’t think this is an improvement, sorry.

@novemberborn
Copy link
Member

Closing due to inactivity. We'd be happy to pick this up again when you have the time to get back to this.

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.

None yet

2 participants