Skip to content
This repository was archived by the owner on Oct 1, 2025. It is now read-only.

chore: test ci reviewer (ignore this)#157

Open
superical wants to merge 2 commits into
masterfrom
test/ci-test-reviewer
Open

chore: test ci reviewer (ignore this)#157
superical wants to merge 2 commits into
masterfrom
test/ci-test-reviewer

Conversation

@superical

Copy link
Copy Markdown
Contributor

Summary

Ignore this. Testing CI reviewer.

Changes

  • Nothing...

Issues

  • Nothing...

Releases

Channels: -

@superical superical changed the title chore: test ci reviewer chore: test ci reviewer (ignore this) Jan 18, 2023
Comment thread tasks/deploy-token.ts
.setAction(async ({ name, symbol, verify, factory, standalone }, hre) => {
const { ethers, network } = hre;
const { contractAddress } = constants;
const {contractAddress} = constants;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🚫 [eslint] <prettier/prettier> reported by reviewdog 🐶
Replace contractAddress with ·contractAddress·

Suggested change
const {contractAddress} = constants;
const { contractAddress } = constants;

Comment thread tasks/deploy-token.ts
const { contractAddress } = constants;
const {contractAddress} = constants;

let testUselessVar;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🚫 [eslint] <no-unused-vars> reported by reviewdog 🐶
'testUselessVar' is defined but never used.

Comment thread tasks/deploy-token.ts Outdated
Comment thread tasks/deploy-token.ts Outdated

let testUselessVar;

var oldSchoolCoolVar = "cool";

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🚫 [eslint] <no-unused-vars> reported by reviewdog 🐶
'oldSchoolCoolVar' is assigned a value but never used.

Comment thread tasks/deploy-token.ts
name,
symbol,
deployer: deployerAddress,
deployer: deployerAddress

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🚫 [eslint] <prettier/prettier> reported by reviewdog 🐶
Insert ,

Suggested change
deployer: deployerAddress
deployer: deployerAddress,

Comment thread tasks/deploy-token.ts
params: [name, symbol, factoryAddress],
contractName,
hre,
hre

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🚫 [eslint] <prettier/prettier> reported by reviewdog 🐶
Insert ,

Suggested change
hre
hre,

Comment thread tasks/deploy-token.ts
constructorArgsParams: [name, symbol, factoryAddress],
contract: "contracts/TradeTrustToken.sol:TradeTrustToken",
hre,
hre

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🚫 [eslint] <prettier/prettier> reported by reviewdog 🐶
Insert ,

Suggested change
hre
hre,

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Comment thread tasks/deploy-token.ts

let testUselessVar;

let oldSchoolCoolVar = "cool";

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🚫 [eslint] <prefer-const> reported by reviewdog 🐶
'oldSchoolCoolVar' is never reassigned. Use 'const' instead.

Suggested change
let oldSchoolCoolVar = "cool";
const oldSchoolCoolVar = "cool";

Comment thread tasks/deploy-token.ts

let testUselessVar;

let oldSchoolCoolVar = "cool";

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🚫 [eslint] <no-unused-vars> reported by reviewdog 🐶
'oldSchoolCoolVar' is assigned a value but never used.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant