Skip to content

Commit 9e9b9dc

Browse files
committed
fix: linting errors
1 parent 2db98f1 commit 9e9b9dc

File tree

4 files changed

+11
-9
lines changed

4 files changed

+11
-9
lines changed

contracts/access/Bouncer.sol

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -171,7 +171,8 @@ contract Bouncer is RBACOwnable {
171171
// them to this contract, which then pings-back to check if it's cool.
172172
// This is also useful for contracts-as-identities: your identity contract implements isValidSignature
173173
// and can then recover the signer and check it against the whitelisted ACTION keys.
174-
if (hasRole(msg.sender, ROLE_DELEGATE) && msg.sender.supportsInterface(InterfaceId_BouncerDelegate)) {
174+
if (hasRole(msg.sender, ROLE_DELEGATE) &&
175+
msg.sender.supportsInterface(InterfaceId_BouncerDelegate)) {
175176
return IBouncerDelegate(msg.sender).isValidSignature(_hash, _sig);
176177
}
177178

contracts/access/BouncerDelegate.sol

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@ pragma solidity 0.4.24;
33
import "./IBouncerDelegate.sol";
44
import "../introspection/SupportsInterfaceWithLookup.sol";
55

6+
67
/**
78
* @title BouncerDelegate
89
* @dev Partial implementation of IBouncerDelegate that adds ERC165 support

contracts/mocks/BouncerDelegateImpl.sol

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@ pragma solidity 0.4.24;
33
import "../access/BouncerDelegate.sol";
44
import "./BouncerMock.sol";
55

6+
67
contract BouncerDelegateImpl is BouncerDelegate {
78

89
bool public shouldPass = false;
@@ -15,12 +16,6 @@ contract BouncerDelegateImpl is BouncerDelegate {
1516
bouncer = _bouncer;
1617
}
1718

18-
function forward()
19-
public
20-
{
21-
bouncer.onlyWithValidSignature("");
22-
}
23-
2419
/**
2520
* @dev verifies that a signature of a hash is valid
2621
* @param _hash message hash that is signed
@@ -37,4 +32,11 @@ contract BouncerDelegateImpl is BouncerDelegate {
3732
{
3833
return shouldPass;
3934
}
35+
36+
function forward()
37+
public
38+
{
39+
bouncer.onlyWithValidSignature("");
40+
}
41+
4042
}

test/introspection/ERC165Checker.test.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,3 @@
1-
import assertRevert from '../helpers/assertRevert';
2-
31
const ERC165Checker = artifacts.require('ERC165CheckerMock');
42
const ERC165NotSupported = artifacts.require('ERC165NotSupported');
53
const ERC165GenerallySupported = artifacts.require('ERC165GenerallySupported');

0 commit comments

Comments
 (0)