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

Expand CONTRIBUTING.md (WIP) #5488

Merged
merged 4 commits into from
Nov 8, 2023
Merged

Conversation

tertsdiepraam
Copy link
Member

@tertsdiepraam tertsdiepraam commented Nov 2, 2023

This is a step towards #5195

The sections "Design Goals" and "Git Etiquette" are still left empty. We can discuss what we what to put there.

@uutils uutils deleted a comment from github-actions bot Nov 7, 2023
Copy link

github-actions bot commented Nov 7, 2023

GNU testsuite comparison:

GNU test failed: tests/tail/symlink. tests/tail/symlink is passing on 'main'. Maybe you have to rebase?

CONTRIBUTING.md Outdated
Write your commit message in the imperative: "Fix bug" and not "Fixed bug"
or "Fixes bug." This convention matches up with commit messages generated
by commands like git merge and git revert.
There are several ways to help and writing code is just one them. Reporting
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
There are several ways to help and writing code is just one them. Reporting
There are several ways to help and writing code is just one of them. Reporting

CONTRIBUTING.md Outdated
to fix yet, make one _before_ you start working on the PR.

Generally, we try to follow what GNU is doing in terms of options and behavior.
It is recommended to look at the GNU Coreutils manual
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
It is recommended to look at the GNU Coreutils manual
It is recommended to look at the GNU coreutils manual

Copy link
Contributor

@cakebaker cakebaker left a comment

Choose a reason for hiding this comment

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

Great work!

@cakebaker cakebaker merged commit d473e45 into uutils:main Nov 8, 2023
42 of 51 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.

2 participants