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

Conversation

@superical
Copy link
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
.setAction(async ({ name, symbol, verify, factory, standalone }, hre) => {
const { ethers, network } = hre;
const { contractAddress } = constants;
const {contractAddress} = constants;

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;

const { contractAddress } = constants;
const {contractAddress} = constants;

let testUselessVar;

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.


let testUselessVar;

var oldSchoolCoolVar = "cool";

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.

name,
symbol,
deployer: deployerAddress,
deployer: deployerAddress

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,

params: [name, symbol, factoryAddress],
contractName,
hre,
hre

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,

constructorArgsParams: [name, symbol, factoryAddress],
contract: "contracts/TradeTrustToken.sol:TradeTrustToken",
hre,
hre

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>

let testUselessVar;

let oldSchoolCoolVar = "cool";

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";


let testUselessVar;

let oldSchoolCoolVar = "cool";

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.

2 participants