Skip to content

Initial revision of documentation for contributors - #13

Merged
JeremyMcCormick merged 1 commit into
mainfrom
iss12
May 19, 2024
Merged

JeremyMcCormick merged 1 commit into
mainfrom
iss12

Conversation

@JeremyMcCormick

Copy link
Copy Markdown
Collaborator

This is the initial version of documentation for contributors.

@molinaro-m molinaro-m left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I skimmed through the text.
I think that it is fine, at least for now.
Maybe a little too much for an entry user, but on Wdnesday we will have feedback on that.

@gmantele gmantele left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Maybe it would useful for the users to know a bit about the repository structure, and especially in which directory are the files to change or add (i.e. content/). I don't think we should invite them to change anything in layout/ for instance.

Comment thread CONTRIBUTING.md Resolved
Comment thread CONTRIBUTING.md

## Install Git

If you do not have it already, then you will need to [install git](https://github.com/git-guides/install-git) in order to work on the website locally.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

There is a simpler alternative that may interest some users not familiar with Git: changes (with branch and PullRequest) can be done online on the GitHub website. This way, there is no need to install Git.

I propose this, but I rarely apply my changes on the GitHub/GitLab website, but I know it is possible. Do you think it would worth saying this is also possible?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Editing files directly in Github seems okay for minor changes like fixing mispellings. I would not encourage it for anything major like adding new pages or major rewrites of pages.

Probably I wouldn't include it in the contributing doc as an option. I would still prefer the PR procedure, even for small changes.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I agree, it is not ideal as not really an as complete solution as a clone of the repository. Although, creating a new page is not that complicating or risky, but ok, let's already start with the clone solution.

Comment thread CONTRIBUTING.md

Hugo is required for building the site locally to check and preview your changes. You may follow the [installation instructions](https://gohugo.io/installation/) for your platform.

A quick procedure to download Hugo on Linux from the [release page](https://github.com/gohugoio/hugo/releases) is:

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

For our website, I wrote a Makefile which lets start a local preview of the current website. Before starting this preview, it checks whether Hugo is installed in the current local directory. If not, it download it from the GitHub Release page and "install" it. This is quite transparent, and also allows me to upgrade Hugo on the users side without having to ask them to do so.

This Makefile works really well on Linux, maybe on MacOS (as it is Unix based ; one may have to test it sometime), not at all on Windows.

Do you think it could be something interesting to share in this repo?

@JeremyMcCormick JeremyMcCormick May 19, 2024 •

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Yes, I think that could be useful. A make build to execute hugo -D would be nice and also maybe make run or something like that to start the server.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'll try to prepare some PR then.

@JeremyMcCormick

Copy link
Copy Markdown
Collaborator Author

Maybe it would useful for the users to know a bit about the repository structure, and especially in which directory are the files to change or add (i.e. content/). I don't think we should invite them to change anything in layout/ for instance.

Agreed - I'll try to add some additional text indicating where the content pages should go.

@JeremyMcCormick

JeremyMcCormick commented May 19, 2024 •

Copy link
Copy Markdown
Collaborator Author

I skimmed through the text.

I think that it is fine, at least for now.

Maybe a little too much for an entry user, but on Wdnesday we will have feedback on that.

I agree that it could be overwhelming for an entry level user. I am thinking that the sections on Git and Github could be streamlined by pointing to some existing external documentation, e.g., the parts on how to make a PR and the git add/commit/push workflow.

It's also possible that the local hugo section can essentially be marked as optional. With the versioned sites working now, it would be possible to just check changes there in the contributor's branch site under /v rather than needing to do it locally.

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