Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Added my day 5 code in Java #361

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

shashwatgupta24
Copy link
Contributor

Please make sure you have done the following:

  • Created a folder with the name of language you code inside directory for the day you are making the contribution to
  • Added the source code file inside /Day<n>/<Language>/
  • Tested your code
  • Added credits (author and date) at the top of the code
  • Updated README.md for /Day <n>/
  • Added yourself as a contributor using all-contributors-cli

@@ -0,0 +1,23 @@
/* @date 25/09/2020
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
/* @date 25/09/2020
/**
* @date 25/09/2020

@@ -0,0 +1,23 @@
/* @date 25/09/2020
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
/* @date 25/09/2020
/**
* @date 25/09/2020

@@ -0,0 +1,33 @@
/* @date 25/09/2020
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
/* @date 25/09/2020
/**
* @date 25/09/2020

@@ -0,0 +1,29 @@
/* @date 25/09/2020
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
/* @date 25/09/2020
/**
* @date 25/09/2020

day5/README.md Outdated
@@ -2158,7 +2158,152 @@ int main() {
return 0;
}
```
/* @date 25/09/2020
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
/* @date 25/09/2020
/**
* @date 25/09/2020

Please add empty line after previous code

}

}
```
Copy link
Member

Choose a reason for hiding this comment

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

Here also


}

```
Copy link
Member

Choose a reason for hiding this comment

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

Same changes here

}
}

```
### Have Another solution?
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
### Have Another solution?
### Have Another solution?

Copy link
Contributor Author

@shashwatgupta24 shashwatgupta24 left a comment

Choose a reason for hiding this comment

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

I have made the requested changes in the date of the programs.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants