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

Updated README file to be more descriptive #141

Merged
merged 3 commits into from
Feb 25, 2024

Conversation

mahamjamal
Copy link
Contributor

ensf400 maham grp5 lab4

changed header ##Invoke to ## Using the Invoke Command

@growlf growlf self-assigned this Feb 25, 2024
@growlf
Copy link
Owner

growlf commented Feb 25, 2024

Your PR description should include all impactful changes, such as the one cat'ing the OS-release file - this is to warn the repo manager of what to expect.

@growlf
Copy link
Owner

growlf commented Feb 25, 2024

All PRs should have an informative description to assist the repo-manager in determining the importance and value of the PR when prioritizing.

@growlf
Copy link
Owner

growlf commented Feb 25, 2024

Additional note: this PR should have been made against the ensf400_w24 branch and not the main branch.

@growlf growlf merged commit 4565532 into growlf:main Feb 25, 2024
1 of 3 checks passed
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.

3 participants