Marked: Coding standards

Created on 27 Feb 2018  路  5Comments  路  Source: markedjs/marked

Marked version: n/a

Proposal type: project operations

What pain point are you perceiving?

More contributions and a lack of defined and documented coding standards.

See also #1082, #1081, #1083, and a few other conversation areas where we've expressed having this pain point.

What solution are you suggesting?

  1. We have lint checks now.
  2. We have Travis now.
  3. We have NPM scripts that fix lint issues for us.
  4. #1081 has an updated checklist for contributing and a list of our NPM scripts and what they do - not merged yet.

What else do we need or want to help contributors know what is expected in their code?

good first issue help wanted proposal question

Most helpful comment

As a new contributor, should I list a few things here that I don't immediately know?

One of the biggest things is whether or not contributors should use ES6/ES7 features (like arrow functions, object spread, filter, map, reduce, etc.).

I tried hacking the code to customize it a bit and I really have no idea what I'm doing despite having two years of experience working with JavaScript. Maybe if someone wrote a doc explaining the underlying mechanics of how the library works newcomers could read it and understand the vision better. This kind of architecture documentation (and indeed an architect who is in charge of the design) is vital to software projects.

All 5 comments

As a new contributor, should I list a few things here that I don't immediately know?

One of the biggest things is whether or not contributors should use ES6/ES7 features (like arrow functions, object spread, filter, map, reduce, etc.).

I tried hacking the code to customize it a bit and I really have no idea what I'm doing despite having two years of experience working with JavaScript. Maybe if someone wrote a doc explaining the underlying mechanics of how the library works newcomers could read it and understand the vision better. This kind of architecture documentation (and indeed an architect who is in charge of the design) is vital to software projects.

No ES6+ until we figure out a transpile build step. We still support older browsers.

That's a fair question...let's go over to #1096

Oh! Since you mentioned architecture, check out #746. Get some idea of where we're planning on going. Fortunately, and unfortunately, we have a large user base; so, we're really trying to satisfy the rest of the zero-major promises, before doing anything drastic.

I think eslint covers this pretty well.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

eGavr picture eGavr  路  4Comments

gclove picture gclove  路  4Comments

priyesh-diukar picture priyesh-diukar  路  3Comments

pigtooter picture pigtooter  路  4Comments

james4388 picture james4388  路  3Comments