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.

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

chore: release build-onchain-apps cli #50

Merged
merged 14 commits into from
Nov 16, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions .changeset/giant-plants-ring.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
'@base-org/build-onchain-apps': patch
---

chore: release CLI
35 changes: 11 additions & 24 deletions .eslintrc.json
Original file line number Diff line number Diff line change
@@ -1,26 +1,8 @@
{
"root": true,
"ignorePatterns": ["**/*"],
"ignorePatterns": ["!**/*"],
"plugins": ["@nx"],
"overrides": [
{
"files": ["*.ts", "*.tsx", "*.js", "*.jsx"],
"rules": {
"@nx/enforce-module-boundaries": [
"error",
{
"enforceBuildableLibDependency": true,
"allow": [],
"depConstraints": [
{
"sourceTag": "*",
"onlyDependOnLibsWithTags": ["*"]
}
]
}
]
}
},
{
"files": ["*.ts", "*.tsx"],
"extends": ["plugin:@nx/typescript"],
Expand All @@ -32,11 +14,16 @@
"rules": {}
},
{
"files": ["*.spec.ts", "*.spec.tsx", "*.spec.js", "*.spec.jsx"],
"env": {
"jest": true
},
"rules": {}
"files": ["*.json"],
"parser": "jsonc-eslint-parser",
"rules": {
"@nx/dependency-checks": [
"error",
{
"ignoredFiles": ["{projectRoot}/vite.config.{js,ts,mjs,mts}"]
}
]
}
}
]
}
3 changes: 2 additions & 1 deletion .github/actions/setup/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,13 +4,14 @@ runs:
steps:
- uses: actions/setup-node@v3
with:
node-version-file: ".nvmrc"
node-version-file: '.nvmrc'
registry-url: https://registry.npmjs.org
cache: yarn

# For provenance https://docs.npmjs.com/generating-provenance-statements#prerequisites
- name: Install npm 9.5
run: npm install -g npm@^9.5.0
shell: bash
Copy link
Contributor Author

Choose a reason for hiding this comment

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


- name: Install Foundry
uses: foundry-rs/foundry-toolchain@v1
Expand Down
33 changes: 17 additions & 16 deletions .github/workflows/node.js.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,9 +5,9 @@
name: Node.js CI
on:
push:
branches: [ "main" ]
branches: ['main']
pull_request:
branches: [ "main" ]
branches: ['main']
jobs:
build:
runs-on: ubuntu-latest
Expand All @@ -16,17 +16,18 @@ jobs:
node-version: [18.x]
# See supported Node.js release schedule at https://nodejs.org/en/about/releases/
steps:
- uses: actions/checkout@v3
- name: Use Node.js ${{ matrix.node-version }}
uses: actions/setup-node@v3
with:
node-version: ${{ matrix.node-version }}
cache: 'npm'
- name: Install dependencies
run: npm install
- name: Format Check
# When fails, please run "npm run format" to your code
run: npm run format:check
- name: Lint Check
# When fails, please run "npm run lint" to your code
run: npm run lint:check
- uses: actions/checkout@v3
- name: Use Node.js ${{ matrix.node-version }}
uses: actions/setup-node@v3
with:
node-version: ${{ matrix.node-version }}
cache: 'npm'
- name: Install dependencies
run: npm install
- name: Lint Check
# When fails, please run "npm run lint" to your code
run: npm run lint
# - name: Format Check
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Format check is breaking the CI. It no longer supported by nx team, see here

# https://github.com/nrwl/nx/issues/4159
# # When fails, please run "npm run format" to your code
# run: npm run format:check --base=main --head=HEAD
8 changes: 4 additions & 4 deletions .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,9 +22,9 @@ jobs:
with:
submodules: recursive

- name: "Setup"
- name: 'Setup'
uses: ./.github/actions/setup

- name: Build
shell: bash
run: yarn build
Expand All @@ -37,8 +37,8 @@ jobs:
id: changesets
uses: changesets/action@v1
with:
title: "chore: version packages 🔖"
commit: "chore: version packages 🔖"
title: 'chore: version packages 🔖'
commit: 'chore: version packages 🔖'
publish: yarn release:publish
version: yarn release:version
env:
Expand Down
5 changes: 0 additions & 5 deletions .gitignore
Original file line number Diff line number Diff line change
@@ -1,9 +1,5 @@
# See http://help.github.com/ignore-files/ for more about ignoring files.

# Nx/Builds
.next
next-env.d.ts

# compiled output
dist
tmp
Expand Down Expand Up @@ -42,5 +38,4 @@ testem.log
.DS_Store
Thumbs.db


.nx/cache
4 changes: 3 additions & 1 deletion .prettierignore
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
# Add files here to ignore them from prettier formatting
/dist
/coverage
/.nx/cache
/templates
/.nx/cache
.github
5 changes: 4 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
Building an Onchain Application can feel overwhelming due to the many libraries, best practices, and choices developers and entrepreneurs must make. **Build Onchain Apps** as a library focuses on automating and simplifying the creation of new Onchain Apps and enhancing existing ones.

We just started; stay tuned for more to come!!! ☕️ 🔵

## Getting Started

More coming here
Expand All @@ -21,6 +21,9 @@ For all available scripts, check the [package.json](https://github.com/base-org/
# Install
yarn

# Format fix
yarn build

# Format fix
yarn format

Expand Down
5 changes: 0 additions & 5 deletions jest.config.ts

This file was deleted.

3 changes: 0 additions & 3 deletions jest.preset.js

This file was deleted.

27 changes: 4 additions & 23 deletions nx.json
Original file line number Diff line number Diff line change
Expand Up @@ -16,21 +16,11 @@
]
},
"e2e": {
"cache": true,
"inputs": ["default", "^production"]
"cache": true
},
"@nx/jest:jest": {
"@nx/vite:test": {
"cache": true,
"inputs": ["default", "^production", "{workspaceRoot}/jest.preset.js"],
"options": {
"passWithNoTests": true
},
"configurations": {
"ci": {
"ci": true,
"codeCoverage": true
}
}
"inputs": ["default", "^production"]
}
},
"namedInputs": {
Expand All @@ -40,17 +30,8 @@
"!{projectRoot}/.eslintrc.json",
"!{projectRoot}/eslint.config.js",
"!{projectRoot}/**/?(*.)+(spec|test).[jt]s?(x)?(.snap)",
"!{projectRoot}/tsconfig.spec.json",
"!{projectRoot}/jest.config.[jt]s",
"!{projectRoot}/src/test-setup.[jt]s",
"!{projectRoot}/test-setup.[jt]s"
"!{projectRoot}/tsconfig.spec.json"
],
"sharedGlobals": []
},
"generators": {},
"pluginsConfig": {
"@nx/js": {
"analyzeSourceFiles": true
}
}
}
57 changes: 25 additions & 32 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,74 +2,67 @@
"name": "@base-org/build-onchain-apps",
alvaroraminelli marked this conversation as resolved.
Show resolved Hide resolved
"version": "0.1.0",
"repository": "https://github.com/base-org/build-onchain-apps.git",
"license": "Apache-2.0",
"dependencies": {
"@nx/devkit": "^17.1.2",
"@nx/js": "^17.1.2",
"chalk": "^4.1.2",
"commander": "^11.1.0",
"got": "^11.8.5",
"figlet": "^1.7.0",
"got": "^11.8.5",
"inquirer": "^8.2.5",
"nx": "^17.1.2",
"rimraf": "^5.0.5",
"tar": "4.4.18",
"tslib": "^2.3.0"
},
"scripts": {
"build": "echo 'Coming soon'",
"format": "nx run-many --target=format --all",
"format:check": "nx run-many --target=format:check --all",
"lint": "nx run-many --target=lint --all",
"lint:check": "nx run-many --target=lint:check --all",
"build": "nx build",
"test": "nx test",
"format": "nx format",
"format:check": "nx format:check",
"lint": "nx lint",
"local:start-registry": "nx local-registry",
"local:publish": "nx publish @base-org/build-onchain-apps",
"release:check": "changeset status --verbose --since=origin/main",
"release:publish": "yarn install && yarn build && changeset publish",
"release:version": "changeset version && yarn install --lockfile-only"
},
"devDependencies": {
"@changesets/changelog-github": "^0.4.8",
"@changesets/cli": "^2.26.2",
"@nx/cypress": "17.1.2",
"@nx/eslint": "17.1.2",
"@nx/eslint-plugin": "17.1.2",
"@nx/jest": "17.1.2",
"@nx/js": "17.1.2",
"@nx/next": "^17.1.2",
"@nx/react": "17.1.2",
"@nx/vite": "17.1.2",
"@nx/workspace": "17.1.2",
"@swc-node/register": "~1.6.7",
"@swc/core": "~1.3.85",
"@testing-library/react": "14.0.0",
"@types/figlet": "^1.5.8",
"@types/inquirer": "^8.2.5",
"@types/jest": "^29.4.0",
"@types/node": "18.14.2",
"@types/react": "18.2.33",
"@types/react-dom": "18.2.14",
"@types/node": "18.7.1",
"@types/tar": "4.0.3",
"@typescript-eslint/eslint-plugin": "^6.9.1",
"@typescript-eslint/parser": "^6.9.1",
"babel-jest": "^29.4.1",
"cypress": "^13.0.0",
"@vitest/coverage-v8": "~0.32.0",
"@vitest/ui": "~0.32.0",
"eslint": "~8.46.0",
"eslint-config-next": "13.4.1",
"eslint-config-prettier": "^9.0.0",
"eslint-plugin-cypress": "^2.13.4",
"eslint-plugin-import": "2.27.5",
"eslint-plugin-jsx-a11y": "6.7.1",
"eslint-plugin-react": "7.32.2",
"eslint-plugin-react-hooks": "4.6.0",
"jest": "^29.4.1",
"jest-environment-jsdom": "^29.4.1",
"jest-environment-node": "^29.4.1",
"nx": "17.1.2",
"prettier": "^2.6.2",
"ts-jest": "^29.1.0",
"ts-node": "10.9.1",
"typescript": "~5.2.2",
"verdaccio": "^5.0.4"
"verdaccio": "^5.27.0",
"vite": "~4.3.9",
"vitest": "~0.32.0"
},
"publishConfig": {
"access": "public",
"provenance": true
},
"nx": {
"includedScripts": []
},
"type": "commonjs",
"main": "./src/index.js",
"typings": "./src/index.d.ts",
"bin": {
"build-onchain-apps": "./src/index.js"
}
}
25 changes: 0 additions & 25 deletions packages/cli/.eslintrc.json

This file was deleted.

25 changes: 0 additions & 25 deletions packages/cli/README.md

This file was deleted.

Loading