-
Notifications
You must be signed in to change notification settings - Fork 62
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
EXPTREE (JULY 17) [CPP] #197
base: master
Are you sure you want to change the base?
Conversation
Hi! Thanks for contributing to this repository. |
@Monal5031 pls check PR #197 let me know if there is still some error |
@aashutoshrathi @Monal5031 pls justify what is the exact error in my PR |
@NishantTanwar The problem is that you have use space indentations while in this repo we follow tabspaces for c and cpp files. Use tab indentation to pass all the tests. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remove the .idea
files.
@@ -0,0 +1,4 @@ | |||
<?xml version="1.0" encoding="UTF-8"?> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remove these .idea
files
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay .. I am updating it
@NishantTanwar Please make changes. |
Yeah. I am updating it. |
Please review @Monal5031 I have updated. |
Remove all unwanted files, make your indentation to tabs and add a new line to the end of the file, as soon as Travis Checks become successful, we will merge it. |
But I guess I have given proper indentation to tabs and also removed the unwanted files |
No, your PR edits 6 files, while it should edit only 3 (including .gitignote). |
@@ -6,6 +6,7 @@ using namespace std; | |||
|
|||
int main() { | |||
|
|||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please undo this change, as it is not part of your PR.
2017/JULY/JULY17/EXPTREE/EXPTREE.cpp
Outdated
|
||
ll modinverse(ll val) { | ||
return exponent(val, MOD - 2); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add a new line at end.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After which line number?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
at the end of file.
@@ -17,5 +17,6 @@ | |||
*.class | |||
*.pyc | |||
*.html | |||
.idea/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rather simply delete the .idea files for now and in another PR you can do this 😉
ping @NishantTanwar Its been 5 days since this PR was last updated, please update it before time. |
@Monal5031 I am not able to get those identation errors hence was not able to update it. I have reviewed my code many times and it seems that I have given tab indentation and not space identation |
@NishantTanwar If you want some help in indenting the code properly you can use
but first you need to install coala using
This will tab indent the code itself. |
@Monal5031 I added a new line at the end but still didn't work. |
@NishantTanwar Did you try indenting the code using coala, because the space indentation error still persists. |
Fixes #194
EXPTREE (JULY 17) [CPP]
.