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.

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

Disable warnings when running in a GitHub Actions workflow #508

Merged
merged 2 commits into from Dec 8, 2020
Merged

Disable warnings when running in a GitHub Actions workflow #508

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
e61e133
Disable warnings when running in a GitHub Actions workflow
sindresorhus Nov 22, 2020
a02742a
Update readme.md
sindresorhus Dec 8, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
4 changes: 4 additions & 0 deletions cli-main.js
View file
Open in desktop
Expand Up @@ -135,6 +135,10 @@ if (typeof options.space === 'string') {
}
}

if (process.env.GITHUB_ACTIONS) {
options.quiet = true;
}

const log = report => {
const reporter = options.reporter || process.env.GITHUB_ACTIONS ? xo.getFormatter(options.reporter || 'compact') : formatterPretty;
process.stdout.write(reporter(report.results));
Expand Down
4 changes: 4 additions & 0 deletions readme.md
View file
Open in desktop
Expand Up @@ -292,6 +292,10 @@ XO will handle the [@typescript-eslint/parser `project` option](https://github.c

See [eslint-config-xo-flow#use-with-xo](https://github.com/xojs/eslint-config-xo-flow#use-with-xo)

## GitHub Actions

XO uses a different formatter when running in a GitHub Actions workflow to be able to get inline annotations. XO also disables warnings here.
Copy link
Member Author

Choose a reason for hiding this comment

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

@fregante Do you know any page I could link to with more info about the inline annotations?

Copy link
Member

@fregante fregante Nov 22, 2020 •
edited

Choose a reason for hiding this comment

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

Found these on their blog


## Config Overrides

XO makes it easy to override configs for specific files. The `overrides` property must be an array of override objects. Each override object must contain a `files` property which is a glob string, or an array of glob strings, relative to the config file. The remaining properties are identical to those described above, and will override the settings of the base config. If multiple override configs match the same file, each matching override is applied in the order it appears in the array. This means the last override in the array takes precedence over earlier ones. Consider the following example:
Expand Down