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

Update assignment2.c #20

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

Parthkrishnan
Copy link

No description provided.

@Phidelux
Copy link
Contributor

Phidelux commented Oct 2, 2020

Hi, looks good to me. However, it would be great if you update the titles of your pull requests to be more self explaining, eg "Assignment 2: Add missing return statement". This title will be used by GitHub to name the merge commit. And if needed add an description to explain why you changed it. This will also help @h0mbre and other reviewers to priorities the reviews.
Thank and regards
Andreas

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.

2 participants