Skip to content

Conversation

@asapekia
Copy link
Contributor

@asapekia asapekia commented Oct 3, 2023

  • I have read CONTRIBUTING.md.
  • This pull request is all my own work -- I have not plagiarized it.
  • All filenames are in PascalCase.
  • All functions and variable names follow Java naming conventions.
  • All new algorithms have a URL in their comments that points to Wikipedia or other similar explanations.
@asapekia
Copy link
Contributor Author

asapekia commented Oct 3, 2023

@vil02 added the steps that you had shared with me for formatting in contribution md

@asapekia
Copy link
Contributor Author

asapekia commented Oct 5, 2023

@vil02 please review

@vil02
Copy link
Member

vil02 commented Oct 7, 2023

@asapekia I am not sure if CONTRIBUTING.md is the right place for such remark.

@siriak what is the right place for such tips and tricks?

@siriak
Copy link
Member

siriak commented Oct 7, 2023

I'd add it as a checkbox in the PR template. This way it is (hopefully) read by everybody opening PRs.

@asapekia
Copy link
Contributor Author

asapekia commented Oct 7, 2023

okay will raise a PR to add it in PR template.

@vil02
Copy link
Member

vil02 commented Oct 7, 2023

okay will raise a PR to add it in PR template.

No! Please do not open a new PR. there are already too many. We can change the title of this one. Everything will be fine.

@asapekia
Copy link
Contributor Author

asapekia commented Oct 7, 2023

okay @vil02 will make the changes in this PR only.

@asapekia asapekia changed the title adding formatting steps in contribution.md adding formatting steps in pull request template Oct 7, 2023
@asapekia
Copy link
Contributor Author

asapekia commented Oct 7, 2023

@vil02 removed the changes from contribution.md and added to pr template. Please review.

Copy link
Member

@siriak siriak left a comment

Choose a reason for hiding this comment

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

Looks good, thanks!

@siriak
Copy link
Member

siriak commented Oct 8, 2023

@vil02 please review

@vil02 vil02 merged commit facc62a into TheAlgorithms:master Oct 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants