Skip to content

Commit c2978e4

Browse files
authored
fix: make it a bit quieter [CLK-321691] (#1)
1 parent 5526b33 commit c2978e4

File tree

2 files changed

+11
-15
lines changed

2 files changed

+11
-15
lines changed

README.md

Lines changed: 9 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1,40 +1,36 @@
11
# octoherd-script-renovate-library-major-version-driver
22

33
<!-- [![@latest](https://img.shields.io/npm/v/@octoherd/script-hello-world.svg)](https://www.npmjs.com/package/@octoherd/script-hello-world) -->
4+
45
[![Build Status](https://github.com/time-loop/octoherd-script-renovate-library-major-version-driver/workflows/Test/badge.svg)](https://github.com/time-loop/octoherd-script-renovate-library-major-version-driver/actions?query=workflow%3ATest+branch%3Amain)
56

67
## Usage
78

89
```bash
910
nvm use v18
1011
node cli.js \
11-
-R time-loop/*-cdk \
12+
-R time-loop/\*-cdk \
1213
-T ghp_0123456789abcdefghijABCDEFGHIJabcdefgh \
1314
--octoherd-bypass-confirms true \
14-
--majorVersion v11
15+
--library @time-loop/cdk-ecs-fargate \
16+
--versionRequirement \>=5.15.2
1517
```
1618

1719
## Options
1820

19-
| option | type | default | description |
20-
| ----------------- | ------ | ------- | ------------------ |
21-
| `--majorVersion` | string | none | major version number for the library, for example v11 |
22-
| `--libray` | string | `@time-loop/cdk-library` | full name of the library to be updated via renovate |
23-
24-
25-
## TODO
26-
27-
- detect and address case where PR is not up-to-date with base branch.
21+
| option | type | default | description |
22+
| ---------------------- | ------ | ------------------------ | --------------------------------------------------- |
23+
| `--versionRequirement` | string | none | requirement for the library, for example `14.*` |
24+
| `--library` | string | `@time-loop/cdk-library` | full name of the library to be updated via renovate |
2825

26+
See [semver](https://www.npmjs.com/package/semver) for details around specifying the `versionRequirement`.
2927

3028
## Limitations
3129

3230
- Should be re-written in TypeScript, but all examples were JS, and we're tight for time.
3331
- Not projen-ified, which is kinda tragic.
3432
I feel that these two technologies are deeply complementary.
3533
https://github.com/projen/projen/issues/2841
36-
- Doesn't differentiate between failed status checks which are required vs optional.
37-
Or... maybe the rollup does? I haven't checked.
3834
- Not published to npmjs.com, so you have to run it locally.
3935
We use github packages, so... we'll probably never publish this to npmjs.com.
4036
Either way, not a priority right now.

script.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ export async function script(
3030
try {
3131
// skip archived repos
3232
if (repository.archived) {
33-
octokit.log.info(`${repository.full_name} is archived, skipping.`);
33+
octokit.log.debug(`${repository.full_name} is archived, skipping.`);
3434
return;
3535
}
3636

@@ -68,7 +68,7 @@ export async function script(
6868
}
6969

7070
if (versions.length < 1) {
71-
octokit.log.error(`${repository.full_name} does not have ${library} in ${path}`);
71+
octokit.log.debug(`${repository.full_name} does not have ${library} in ${path}`);
7272
return;
7373
}
7474

0 commit comments

Comments
 (0)