Skip to content

Commit

Permalink
Merge #1960
Browse files Browse the repository at this point in the history
1960: fix(info): test if logLocation is writable r=ZauberNerd a=ZauberNerd


<details>
<summary>Bors merge bot cheat sheet</summary>

We are using [bors-ng](https://github.com/bors-ng/bors-ng) to automate merging of our pull requests. The following table provides a summary of commands that are available to reviewers (members of this repository with push access) and delegates (in case of `bors delegate+` or `bors delegate=[list]`).

| Syntax | Description |
| --- | --- |
| bors merge | Run the test suite and push to master if it passes. Short for "reviewed: looks good." |
| bors merge- | Cancel an r+, r=, merge, or merge= |
| bors try | Run the test suite without pushing to master. |
| bors try- | Cancel a try |
| bors delegate+ | Allow the pull request author to merge their changes. |
| bors delegate=[list] | Allow the listed users to r+ this pull request's changes. |
| bors retry | Run the previous command a second time. |

This is a short collection of opinionated commands. For a full list of the commands read the [bors reference](https://bors.tech/documentation/).

</details>


Co-authored-by: Björn Brauer <bjoern.brauer@new-work.se>
  • Loading branch information
bors[bot] and ZauberNerd authored Sep 7, 2021
2 parents 7a7a69b + 6ba3261 commit 37fcfd5
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion packages/info/logger/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,9 @@ class Logger {
this.level = logLevels.info;

this.logLocation = process.env.HOPS_LOG_LOCATION;
if (this.logLocation && !accessSync(this.logLocation, constants.W_OK)) {
try {
accessSync(this.logLocation, constants.W_OK);
} catch {
this.logLocation = '';
}

Expand Down

0 comments on commit 37fcfd5

Please sign in to comment.