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

ACL: Fix inline doc and functions indentation #579

Merged
merged 2 commits into from
May 29, 2020
Merged

Conversation

facuspagnuolo
Copy link
Contributor

No description provided.

@@ -32,7 +32,6 @@ contract ACL is IACL, TimeHelpers, AragonApp, ACLHelpers {
uint8 internal constant ORACLE_PARAM_ID = 203;
uint8 internal constant LOGIC_OP_PARAM_ID = 204;
uint8 internal constant PARAM_VALUE_PARAM_ID = 205;
// TODO: Add execution times param type?
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Moved here #580

Copy link

@Sweetilona3 Sweetilona3 May 5, 2023

Choose a reason for hiding this comment

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

Well see!😉🤑

@auto-assign auto-assign bot requested a review from izqui May 27, 2020 19:23
@coveralls
Copy link

Coverage Status

Coverage remained the same at 99.115% when pulling bb13e35 on fix_acl_inline_doc into d911a8b on next.

Copy link
Contributor

@sohkai sohkai left a comment

Choose a reason for hiding this comment

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

LGTM!

@sohkai sohkai merged commit 8d40fea into next May 29, 2020
@sohkai sohkai deleted the fix_acl_inline_doc branch May 29, 2020 13:41
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.

5 participants