Skip to content

Add a CONTRIBUTING.md guide (and an example on the README)#1315

Open
antoine-levitt wants to merge 2 commits into
JuliaMolSim:masterfrom
antoine-levitt:contributing
Open

Add a CONTRIBUTING.md guide (and an example on the README)#1315
antoine-levitt wants to merge 2 commits into
JuliaMolSim:masterfrom
antoine-levitt:contributing

Conversation

@antoine-levitt
Copy link
Copy Markdown
Member

No description provided.

Copy link
Copy Markdown
Collaborator

@Technici4n Technici4n left a comment

Choose a reason for hiding this comment

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

Indeed, the AI clause is now unfortunately necessary.

Copy link
Copy Markdown
Member

@mfherbst mfherbst left a comment

Choose a reason for hiding this comment

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

I agree with starting the contributing guide. See my comment on the example, though.

Comment thread README.md
Comment thread CONTRIBUTING.md
(relatively easy tasks suitable for newcomers) or [help wanted](https://github.com/JuliaMolSim/DFTK.jl/issues?q=is%3Aissue+is%3Aopen+label%3A%22help+wanted%22)
(more sizeable but well-defined and isolated).
Don't hesitate to ask for help, through github,
email, [JuliaMolSim zulip][chat-url] or [Matrix chat][matrix-url].
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.

This is likely broken

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Do we remove both?

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.

Just paste the actual urls from the README ?

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