@pablobm commented on this pull request.
> + your needs well, and frames them in the context of the wider community.
+- **Code reviews:** at any given point, there will be pending PRs, waiting for
+ reviews. Can you take on one, understand what it's trying to do, and
+ provide actionable feedback? Is the code clear, maintainable, and readable?
+ Would you do something differently? Are useful, clear tests provided?
+- **Code:** take an existing issue and try to fix it, or try to implement
+ an idea.
+- And probably other ideas not captured here.
+
+Bear in mind that OSM attracts very diverse contributors with very different
+needs. Others may have needs different from yours, and reaching a consensus
+is sometimes difficult.
+
+If you want to code a feature to this repository, we recommend that you ask
+for feedback early and often. Create an issue to discuss it, or start with a
+Draft PR that shows your intention clearly and can be used to provide early
Good catch. Initially I capitalised it on purpose as I am referring to the
specific feature to create "draft" PRs.
On second thought, that might be too implicit for new contributors. I have now
changed style:
- All instances are de-capitalised.
- The first instance is in quotation marks and links to an explanation of what
it refers to.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6543#discussion_r2564584056
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/6543/review/[email protected]>
_______________________________________________
rails-dev mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/rails-dev
