Skip to content

refactor: port is-ignored ts#675

Merged
marionebl merged 6 commits intomasterfrom
refactor-port-is-ignored-ts
Jun 11, 2019
Merged

refactor: port is-ignored ts#675
marionebl merged 6 commits intomasterfrom
refactor-port-is-ignored-ts

Conversation

@marionebl
Copy link
Copy Markdown
Contributor

@marionebl marionebl mentioned this pull request Jun 2, 2019
18 tasks
@marionebl marionebl requested review from byCedric and escapedcat June 3, 2019 11:37
@@ -0,0 +1 @@
export * from "./lib"; No newline at end of file
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.

What's better, double or single quotes? In other files I see that you are using single, I prefer them too but open for double. 😄

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 vote single :P

Comment thread @commitlint/is-ignored/src/defaults.ts Outdated
@@ -0,0 +1,152 @@
import isIgnored from './is-ignored';

const VERSION_MESSAGES = [
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.

Love the all caps constants 😄

'0.0.1-some-crazy-tag.999',
'0.0.1-1e69d54',
'v0.0.1',
' v3.0.0'
Copy link
Copy Markdown
Member

@byCedric byCedric Jun 3, 2019

Choose a reason for hiding this comment

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

Just a check, is the extra space intentional?

@marionebl marionebl merged commit 71928ae into master Jun 11, 2019
@marionebl marionebl deleted the refactor-port-is-ignored-ts branch June 11, 2019 09:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants