Skip to content

fix: typo 'nonrebase_cotract' -> 'nonrebase_contract' in tests#2744

Open
phessophissy wants to merge 1 commit intoOriginProtocol:masterfrom
phessophissy:fix/typo-nonrebase-contract
Open

fix: typo 'nonrebase_cotract' -> 'nonrebase_contract' in tests#2744
phessophissy wants to merge 1 commit intoOriginProtocol:masterfrom
phessophissy:fix/typo-nonrebase-contract

Conversation

@phessophissy
Copy link

Fixes typo in test variable names where "contract" was misspelled as "cotract" (missing 'n').

Files changed:

  • contracts/test/_fixture.js (28 occurrences)
  • contracts/test/token/ousd.js (5 occurrences)
  • contracts/test/token/token-transfers.js (24 occurrences)

Variables renamed:

  • nonrebase_cotract_0nonrebase_contract_0
  • nonrebase_cotract_1nonrebase_contract_1
  • nonrebase_cotract_notSet_0nonrebase_contract_notSet_0
  • nonrebase_cotract_notSet_1nonrebase_contract_notSet_1
  • nonrebase_cotract_notSet_altcpt_gt_0nonrebase_contract_notSet_altcpt_gt_0
  • nonrebase_cotract_notSet_altcpt_gt_1nonrebase_contract_notSet_altcpt_gt_1

@naddison36
Copy link
Collaborator

We have some big PRs waiting to be merged into master so we'll hold off merging this trivial PR until the high priority ones are merged.

Copy link
Collaborator

@naddison36 naddison36 left a comment

Choose a reason for hiding this comment

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

Still good post cleanup

@naddison36
Copy link
Collaborator

@phessophissy can merge with OriginProtocol:master so we have a fresh build

Thanks

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants