Skip to content

Minor standards and cleanup additions. #581

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.

Sign up for GitHub

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

Merged
frangio merged 9 commits into from
Dec 1, 2017
Merged

Minor standards and cleanup additions. #581

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
887c1bb
feat: more comprehensive gitignore
shrugs Nov 24, 2017
b9b26e1
feat: allow configuration of ropsten and ganache with env variables
shrugs Nov 24, 2017
6ea0577
feat: enable eslint
shrugs Nov 24, 2017
8662846
feat: apply eslint --fix across project
shrugs Nov 24, 2017
58abd66
feat: rename all test files to include .test.js postfix
shrugs Nov 24, 2017
227a335
feat: move mock solidity contracts to mocks/ folder
shrugs Nov 24, 2017
d07bdcc
feat: manually fix linting errors across tests
shrugs Nov 24, 2017
258c001
feat: add linting to travis script
shrugs Nov 27, 2017
ac38088
fix: default to empty string for mnemonic
shrugs Dec 1, 2017
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
Prev Previous commit
Next Next commit
feat: apply eslint --fix across project
  • Loading branch information
@shrugs
shrugs committed Dec 1, 2017
commit 86628468386b2ed19b74101fa65529b45bdb8b8c
2 changes: 2 additions & 0 deletions .eslintrc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,8 @@
// Code style
"indent": [2, 2],
"quotes": [2, "single"],
"semi": ["error", "always"],
"space-before-function-paren": ["error", "always"],
"no-use-before-define": 0,
"eqeqeq": [2, "smart"],
"dot-notation": [2, {"allowKeywords": true, "allowPattern": ""}],
Expand Down
4 changes: 2 additions & 2 deletions migrations/1_initial_migration.js
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
var Migrations = artifacts.require("./Migrations.sol");
var Migrations = artifacts.require('./Migrations.sol');

module.exports = function(deployer) {
module.exports = function (deployer) {
deployer.deploy(Migrations);
};
6 changes: 3 additions & 3 deletions migrations/2_deploy_contracts.js
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,9 +1,9 @@
//var Ownable = artifacts.require("ownership/Ownable.sol");
// var Ownable = artifacts.require("ownership/Ownable.sol");

// NOTE: Use this file to easily deploy the contracts you're writing.
// (but make sure to reset this file before committing
// with `git checkout HEAD -- migrations/2_deploy_contracts.js`)

module.exports = function(deployer) {
//deployer.deploy(Ownable);
module.exports = function (deployer) {
// deployer.deploy(Ownable);
};
Loading