Skip to content
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.

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

Multiple sources paths, rejected transaction error message #48

Merged
merged 9 commits into from
Feb 1, 2022

Conversation

dribeiro-ShardLabs
Copy link
Contributor

@dribeiro-ShardLabs dribeiro-ShardLabs commented Jan 27, 2022

  • Multiple paths are allowed now for hardhat.config file.
  • Error message from a rejected transaction now displayed.

@dribeiro-ShardLabs dribeiro-ShardLabs changed the title Added support for multiple sources paths Added support for multiple sources paths, displaying error message of a rejected transaction Jan 27, 2022
@dribeiro-ShardLabs dribeiro-ShardLabs changed the title Added support for multiple sources paths, displaying error message of a rejected transaction Multiple sources paths, rejected transaction error message Jan 27, 2022
src/types.ts Outdated Show resolved Hide resolved
src/types.ts Outdated Show resolved Hide resolved
resolve();
},
error => {
console.log(`Invoke transaction ${txHash} is REJECTED`);
Copy link
Collaborator

Choose a reason for hiding this comment

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

Would it make sense to use console.error here?

Copy link
Collaborator

Choose a reason for hiding this comment

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

This remained, do we keep this as it is?

src/task-actions.ts Outdated Show resolved Hide resolved
src/task-actions.ts Outdated Show resolved Hide resolved
@FabijanC FabijanC self-requested a review February 1, 2022 17:02
resolve();
},
error => {
console.log(`Invoke transaction ${txHash} is REJECTED`);
Copy link
Collaborator

Choose a reason for hiding this comment

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

This remained, do we keep this as it is?

cairoPaths.push(path);
});
}
if (hre.config.paths.cairoPaths) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

This should be documented in Readme

@dribeiro-ShardLabs dribeiro-ShardLabs merged commit 70cb009 into master Feb 1, 2022
@FabijanC FabijanC deleted the multiple-cairo-paths-config-file branch May 6, 2022 12:04
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.

None yet

2 participants