Skip to content

Commit

Permalink
chore: cleanups
Browse files Browse the repository at this point in the history
  • Loading branch information
ealush committed Oct 1, 2023
1 parent 3caa744 commit ca735df
Show file tree
Hide file tree
Showing 8 changed files with 25 additions and 19 deletions.
8 changes: 7 additions & 1 deletion .eslintrc.js
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,13 @@ module.exports = {
rules: {
'@typescript-eslint/no-explicit-any': 'off',
'@typescript-eslint/explicit-module-boundary-types': 'off',
'@typescript-eslint/no-unused-vars': 'warn',
'@typescript-eslint/no-unused-vars': [
1,
{
argsIgnorePattern: '^_',
varsIgnorePattern: '^_',
},
],
},
},
{
Expand Down
15 changes: 8 additions & 7 deletions .github/workflows/integration.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,19 +28,19 @@ jobs:
- uses: actions/checkout@v3
with:
fetch-depth: 0
- name: Setup .yarnrc.yml
run: |
yarn config set npmAuthToken $NPM_TOKEN || echo "No NPM_TOKEN found"
yarn config set npmAlwaysAuth true
echo //registry.npmjs.org/:_authToken=$NPM_TOKEN > ~/.npmrc || echo "No NPM_TOKEN found"
env:
NPM_TOKEN: ${{ secrets.NPM_PUBLISH_TOKEN }}
- name: Install
run: yarn install --immutable
- name: Test
run: yarn test
- name: Lint
run: yarn lint
- name: Setup .yarnrc.yml
run: |
yarn config set npmAuthToken $NPM_TOKEN
yarn config set npmAlwaysAuth true
echo //registry.npmjs.org/:_authToken=$NPM_TOKEN > ~/.npmrc
env:
NPM_TOKEN: ${{ secrets.NPM_PUBLISH_TOKEN }}
- name: Release
run: yarn release
env:
Expand All @@ -54,4 +54,5 @@ jobs:
PUBLIC_REPO_TOKEN: ${{secrets.PUBLIC_REPO_TOKEN}}
EMAIL_ADDRESS: ${{secrets.EMAIL_ADDRESS}}
GIT_NAME: ${{secrets.GIT_NAME}}
NPM_TOKEN: ${{ secrets.NPM_PUBLISH_TOKEN }}
npmAuthToken: ${{secrets.NPM_PUBLISH_TOKEN}}
6 changes: 1 addition & 5 deletions .yarnrc.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,12 +2,8 @@ nmMode: hardlinks-local

nodeLinker: node-modules

npmAlwaysAuth: true

npmAuthToken: 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

plugins:
- path: .yarn/plugins/@yarnpkg/plugin-interactive-tools.cjs
spec: "@yarnpkg/plugin-interactive-tools"
spec: '@yarnpkg/plugin-interactive-tools'

yarnPath: .yarn/releases/yarn-3.5.1.cjs
2 changes: 1 addition & 1 deletion packages/vest/src/core/isolate/IsolateReconciler.ts
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import { TIsolate } from 'vestjs-runtime';

export abstract class IsolateReconciler {
static match(currentNode: TIsolate, historyNode: TIsolate): boolean {
static match(_currentNode: TIsolate, _historyNode: TIsolate): boolean {
return false;
}

Expand Down
Original file line number Diff line number Diff line change
@@ -1,9 +1,9 @@
import { IsolateReconciler } from 'IsolateReconciler';
import { Maybe, deferThrow, text } from 'vest-utils';
import { IsolateInspector, Reconciler } from 'vestjs-runtime';
import type { TIsolate } from 'vestjs-runtime';

import { ErrorStrings } from 'ErrorStrings';
import { IsolateReconciler } from 'IsolateReconciler';
import type { TIsolateTest } from 'IsolateTest';
import { VestTest } from 'VestTest';
import cancelOverriddenPendingTest from 'cancelOverriddenPendingTest';
Expand Down
3 changes: 1 addition & 2 deletions packages/vest/src/testUtils/partition.ts
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
export default function partition<T>(
array: T[],
// eslint-disable-next-line no-unused-vars
predicate: (value: T, index: number, array: T[]) => boolean
predicate: (value: T, _index: number, array: T[]) => boolean
): [T[], T[]] {
return array.reduce(
(partitions: [T[], T[]], value, number) => {
Expand Down
4 changes: 2 additions & 2 deletions packages/vest/src/testUtils/suiteDummy.ts
Original file line number Diff line number Diff line change
Expand Up @@ -90,14 +90,14 @@ export function untested(fields?: OneOrMoreOf<string>) {

function createSuiteRunResult(
fieldNames: Maybe<string[] | string>,
callback: (fieldName?: string) => void // eslint-disable-line no-unused-vars
callback: (_fieldName?: string) => void
) {
return createSuite(fieldNames, callback)();
}

function createSuite(
fieldNames: Maybe<string[] | string> = 'field_1',
callback: (fieldName?: string) => void // eslint-disable-line no-unused-vars
callback: (_fieldName?: string) => void
) {
return create(() => {
asArray(fieldNames).forEach(fieldName => callback(fieldName));
Expand Down
4 changes: 4 additions & 0 deletions vx/scripts/release/steps/push_to_latest_branch.sh
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,10 @@ git checkout -b $STABLE_BRANCH
echo "Rebuilding yarn.lock"
yarn

echo "Cleaning up auth token"
yarn config unset npmAuthToken
yarn config unset npmAlwaysAuth

echo "Commiting"
git add -A
git commit -m "$1" -m "$2"
Expand Down

2 comments on commit ca735df

@vercel
Copy link

@vercel vercel bot commented on ca735df Oct 1, 2023

Choose a reason for hiding this comment

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

Successfully deployed to the following URLs:

vest-next – ./website

vest-website.vercel.app
vest-next-ealush.vercel.app
vest-next.vercel.app
vest-next-git-latest-ealush.vercel.app

@vercel
Copy link

@vercel vercel bot commented on ca735df Oct 1, 2023

Choose a reason for hiding this comment

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

Successfully deployed to the following URLs:

vest – ./website

vestjs.dev
vest-ealush.vercel.app
vest.vercel.app
vest-git-latest-ealush.vercel.app
www.vestjs.dev

Please sign in to comment.