Skip to content

Update CONTRIBUTING.md#3924

Merged
hdgarrood merged 1 commit intopurescript:masterfrom
hdgarrood:update-contributing-md
Sep 1, 2020
Merged

Update CONTRIBUTING.md#3924
hdgarrood merged 1 commit intopurescript:masterfrom
hdgarrood:update-contributing-md

Conversation

@hdgarrood
Copy link
Contributor

  • Link to governance doc where appropriate
  • Remove reference to core-tests (which no longer exist)
  • Try to set expectations a bit better re: approving proposals or
    merging PRs
  • Rearrange the sections a little for clarity
  • Mention the make task so that it's easier to regenerate the LICENSE
    file

@rhendric was asking me about this just now and I thought it would be a good opportunity to have this written down somewhere people can see it, as well as give this file a much-needed update.

* Link to governance doc where appropriate
* Remove reference to core-tests (which no longer exist)
* Try to set expectations a bit better re: approving proposals or
  merging PRs
* Rearrange the sections a little for clarity
* Mention the `make` task so that it's easier to regenerate the LICENSE
  file
@hdgarrood hdgarrood merged commit ba55bb1 into purescript:master Sep 1, 2020
@hdgarrood hdgarrood deleted the update-contributing-md branch September 1, 2020 19:22
Comment on lines +54 to +55
[the #purescript channel in FP Slack]: https://functionalprogramming.slack.com/
[Discourse]: https://discourse.purescript.org/
Copy link
Contributor

Choose a reason for hiding this comment

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

Are these two lines supposed to be here? They don't appear in the rendered markdown.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes: they allow you to write e.g. [the #purescript channel in FP Slack][] anywhere in the doc without having to have the link destination inline. Removing them would break the links earlier up in the doc.

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

Comments