Skip to content

Commit

Permalink
fix!: deprecate and remove store-aws related code
Browse files Browse the repository at this point in the history
BREAKING CHANGE: Deprecate and remove `store-aws` related code.
  • Loading branch information
sf-v committed Nov 12, 2023
1 parent 8debc71 commit 0d63443
Show file tree
Hide file tree
Showing 9 changed files with 21 additions and 736 deletions.
2 changes: 1 addition & 1 deletion docs/ci/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,7 @@ storing the artifacts on AWS.
To use an external storage provider, pass the `--externalStorage` flag:

```sh
best --compareStats ${BASE_COMMIT} ${TARGET_COMMIT} --externalStorage=@best/store-aws
best --compareStats ${BASE_COMMIT} ${TARGET_COMMIT}
```

You must also set the `AWS_BUCKET_NAME` environment variable so Best
Expand Down
17 changes: 0 additions & 17 deletions docs/configuration/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -56,12 +56,6 @@ config files or a local repo with Best configured.

`boolean` Displays calculated `globalConfig` and project configs.

### `--externalStorage`

`string` Enables saving the results in an arbitrary storage system.
Provide the external storage adapter to use. Best supports AWS with
the `@best/store-aws` adapter.

### `--compareStats`

`array` Compares the benchmarks of two commits against each other.
Expand Down Expand Up @@ -125,12 +119,6 @@ options.

When best is not run on the main branch the snapshot is marked as temporary.

### `externalStorage`

`string` Allows saving the results in an arbitrary storage system.
Specify the external storage adapter to use. Currently Best supports
AWS with the `@best/store-aws` adapter.

### `metrics`

`array` The set of metrics to capture. Currently supported options
Expand Down Expand Up @@ -319,8 +307,3 @@ a value for `TARGET_COMMIT` as well.

`string` The target commit to be used with `--compareStats`. Set
a value for `BASE_COMMIT` as well.

### `AWS_BUCKET_NAME`

`string` Name of the AWS bucket to use when `--externalStorage=@best/store-aws`
is provided.
27 changes: 0 additions & 27 deletions packages/@best/store-aws/package.json

This file was deleted.

157 changes: 0 additions & 157 deletions packages/@best/store-aws/src/aws-wrapper.ts

This file was deleted.

91 changes: 0 additions & 91 deletions packages/@best/store-aws/src/index.ts

This file was deleted.

10 changes: 0 additions & 10 deletions packages/@best/store-aws/tsconfig.json

This file was deleted.

10 changes: 0 additions & 10 deletions packages/@best/store-aws/typings/node-fetch.d.ts

This file was deleted.

1 change: 0 additions & 1 deletion tsconfig.json
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@
{ "path": "./packages/@best/runtime" },
{ "path": "./packages/@best/shared" },
{ "path": "./packages/@best/store" },
{ "path": "./packages/@best/store-aws" },
{ "path": "./packages/@best/store-fs" },
{ "path": "./packages/@best/frontend" },
{ "path": "./packages/@best/api-db" }
Expand Down
Loading

0 comments on commit 0d63443

Please sign in to comment.