You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
#24832 proposes that we auto-wrap commit messages when importing from GitHub.
It would probably also be a good idea to document our wrapping conventions: I don't currently see that detail on https://golang.org/doc/contribute.html#commit_changes, although the whole page is a bit difficult to skim as a reference anyway.
I initially had some difficulty with the package-prefix convention too: more examples would probably be helpful there. For example, it's not obvious what to do with the x/ repos, or packages with nested paths, or things that are not importable Go packages.
(One can discover those conventions by reading the change history, but that seems like a high bar for a first-time contributor who may well just be fixing a typo in a doc comment.)
The text was updated successfully, but these errors were encountered:
#24832 proposes that we auto-wrap commit messages when importing from GitHub.
It would probably also be a good idea to document our wrapping conventions: I don't currently see that detail on https://golang.org/doc/contribute.html#commit_changes, although the whole page is a bit difficult to skim as a reference anyway.
I initially had some difficulty with the package-prefix convention too: more examples would probably be helpful there. For example, it's not obvious what to do with the
x/
repos, or packages with nested paths, or things that are not importable Go packages.(One can discover those conventions by reading the change history, but that seems like a high bar for a first-time contributor who may well just be fixing a typo in a doc comment.)
The text was updated successfully, but these errors were encountered: