Skip to content

Commit

Permalink
chore: remove sandbox and cli npm pkgs (#3567)
Browse files Browse the repository at this point in the history
Removes publishing of sandbox and cli npm packages
Also removes its references from the code (docs updates were already
done)

1. Do we think we should wait a bit before completely nuking the npm
section of the aztec-cli script? Currently i have just added deprecation
messages
2. I couldn't remove `@aztec/cli` npm package from `end-to-end` because
it uses `getProgram()`. Lmk if anyone has any good ideas
  • Loading branch information
rahul-kothari authored Jan 12, 2024
1 parent 80c3805 commit a8cf1bf
Show file tree
Hide file tree
Showing 5 changed files with 2 additions and 173 deletions.
2 changes: 1 addition & 1 deletion boxes/token/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

This is a minimal [Aztec](https://aztec.network/) Noir smart contract and frontend bootstrapped with [`aztec-cli unbox`](https://github.com/AztecProtocol/aztec-packages/tree/master/yarn-project/cli). It is recommended you use the `aztec-cli unbox PrivateToken` command so that the repository is copied with needed modifications from the monorepo subpackage.

Some contract specific settings for `PrivateToken` are in a [config](src/config.ts) will require manual updates depending on your changes to the source code. `aztec-cli` can be installed with `npm i -g @aztec/cli`, if you don't have it already.
Some contract specific settings for `PrivateToken` are in a [config](src/config.ts) will require manual updates depending on your changes to the source code.

## Setup

Expand Down
163 changes: 0 additions & 163 deletions yarn-project/cli/aztec-cli

This file was deleted.

6 changes: 0 additions & 6 deletions yarn-project/cli/aztec-cli-dev

This file was deleted.

2 changes: 1 addition & 1 deletion yarn-project/cli/src/update/npm.ts
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ import { SemVer, parse } from 'semver';
import { atomicUpdateFile } from '../utils.js';
import { DependencyChanges } from './common.js';

const deprecatedNpmPackages = new Set<string>([]);
const deprecatedNpmPackages = new Set<string>(['@aztec/cli', '@aztec/aztec-sandbox']);
const npmDeprecationMessage = `
The following packages have been deprecated and will no longer be updated on the npm registry:
${Array.from(deprecatedNpmPackages)
Expand Down
2 changes: 0 additions & 2 deletions yarn-project/deploy_npm.sh
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,6 @@ deploy_package l1-artifacts
deploy_package ethereum
deploy_package noir-compiler
deploy_package noir-contracts
deploy_package cli
deploy_package merkle-tree
deploy_package noir-protocol-circuits
deploy_package acir-simulator
Expand All @@ -100,4 +99,3 @@ deploy_package p2p
deploy_package world-state
deploy_package sequencer-client
deploy_package aztec-node
deploy_package aztec-sandbox

0 comments on commit a8cf1bf

Please sign in to comment.