Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

refactor: evaluate RuleElements with @sindresorhus/ow#isValid #101

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

refactor: evaluate RuleElements with @sindresorhus/ow#isValid #101

wants to merge 6 commits into from

Conversation

gregswindle
Copy link
Contributor

@gregswindle gregswindle commented Aug 27, 2019 •
edited
Loading

git Pull Request Pull Request (PR)

  1. Refactor all RuleElement operator methods to use @sindresorhus/ow#isValid.
  2. Remove unused development and production dependencies.
  3. Update all third-party dependencies to their latest versions.
  4. Rename archetypes-rules => @archetypes/rules.

Associated issue(s)

Closes #97
Closes #103

  • 1. The acceptance criteria for all associated issues have been
    completed, tested, and validated.
  • 2. The README either reflects my changes or does not require updates.
  • 3. The CONTRIBUTING document either reflects my changes or does not
    require updates.
View the "Code Review Cheat-sheet"...

Rule 1: Review all Code before Releasing it

Do the code reviews before deployment.

Why:

Your team will end up, on average, spending 7% more of its time on building
new features compared with those who do after, and 10% more than those who
don’t do code reviews at all.

Rule 2: All developers review code

Make sure all your developers get to review code.

Why:

This will improve the feeling of empowerment, facilitate knowledge transfer,
and improve developer satisfaction and productivity.

Rule 3: Four-to-Eight (4-8) Hours per Developers per Week

The optimal amount of time to spend on code reviews is between 0.5 to 1 day per
week per developer.

Rule 4: Don't Release Code that Fails Reviews

Make code reviews blocking, that is, don’t deploy before they have been carried
out.

Rule 5: More Rigor, Better Quality and Velocity

Be strict and thorough when reviewing code.

Why:

Your code quality and velocity will thank you.

Code quality summary

Measure Scores
Quality gate Sonar Alert Status Metrics
Duplications Sonar Duplicated Lines Density Metrics
Maintainability Sonar Code Smells Metrics
Sonar Sqale Rating Metrics
Sonar Sqale Index Metrics
Sonar Ncloc Metrics
Reliability Sonar Reliability Rating Metrics
Sonar Bugs Metrics
Security Sonar Security Rating Metrics
Sonar Vulnerabilities Metrics
Test coverage Sonar Coverage Metrics

Code quality, vulnerability, and standards compliance tools

Code Style Linters Test frameworks
JavaScript Style Guide ESLint Jest BDD
Standard JS user guide link-external ESlint user guide link-external Jest user guide link-external

How to format, lint, and test your changes

Open a Terminal, go to the root directory for archetypes-rules, and run:

$ npm test

info Completed tasks are not required to open a PR, and may
be addressed while the PR is open.

alert All tasks must be completed and verified
before a PR may be merged into master, however.

How to test a release

The Pre-release test instructions include step-by-step guidelines for bundling, packing, and testing this module as it would be released on NPM.

@gregswindle
fix(package): update dependencies
87fc615 
Production

1.  @sindresorhus/[email protected]
2.  [email protected]

Development

1.  @semantic-release/[email protected]
2.  @semantic-release/[email protected]
3.  @semantic-release/[email protected]
4.  @semantic-release/[email protected]
5.  @semantic-release/[email protected]
6.  @semantic-release/[email protected]
7.  [email protected]
8.  [email protected]
9.  [email protected]
10. [email protected]
11. [email protected]
12. [email protected]
13. [email protected]
14. [email protected]
15. [email protected]
16. [email protected]
17. [email protected]
18. [email protected]
19. [email protected]
20. [email protected]
21. [email protected]
22. [email protected]
23. [email protected]
24. [email protected]
25. [email protected]
26. [email protected]
27. [email protected]
28. [email protected]
29. [email protected]
30. [email protected]
31. [email protected]
32. [email protected]
33. [email protected]
34. [email protected]
35. [email protected]
36. [email protected]
37. [email protected]
38. [email protected]
39. [email protected]
40. [email protected]
41. [email protected]
42. [email protected]

Run

- npm audit fix
- npm prune
- npm dedupe

Closes #57
Closes #66
Closes #69
Closes #70
Closes #72
Closes #80
Closes #84
Closes #87
Closes #95
Closes #99
@gregswindle
chore(package): clean dependencies
0643ce9 
Update dev-dependencies:

1.  [email protected] --save-dev
2.  [email protected] --save-dev
3.  [email protected] --save-dev

Remove unused dev and prod dependencies.

\#97
@gregswindle gregswindle self-assigned this Aug 27, 2019
@gregswindle gregswindle added type: breaking change Modifications affect or replace the public API. ⇧ Bumps the MAJOR semver. type: dependency Update or pin a third-party library. ⇧ Bump the PATCH semver. type: refactor Restructuring source code without changing its external behavior. type: test Adding, correcting, or improving tests (coverage). labels Aug 27, 2019
codeclimate[bot]
codeclimate bot reviewed Aug 27, 2019
View reviewed changes
Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

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

The PR diff size of 41721 lines exceeds the maximum allowed for the inline comments feature.

codeclimate[bot]
codeclimate bot reviewed Sep 7, 2019
View reviewed changes
Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

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

The PR diff size of 43253 lines exceeds the maximum allowed for the inline comments feature.

@commonality commonality deleted a comment Sep 7, 2019
@gregswindle
refactor(package): replace lodash.hasin
035db0f 
Replace module lodash.hasin (4 KB) with private
function objectHasPropertyName (500 B)

#97
@gregswindle
refactor(method-count): remove extra functions
c1e412f 
- Remove ArrayVariable.prototype.isNonEmpty
- Rename Proposition results' names to match their
  operator method names.

#103
@gregswindle
refactor(function): move function up
b8629f2 
Move up function assignTypeTo for reuse.

#97
codeclimate[bot]
codeclimate bot reviewed Sep 7, 2019
View reviewed changes
Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

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

The PR diff size of 43356 lines exceeds the maximum allowed for the inline comments feature.

@codeclimate
Copy link

codeclimate bot commented Sep 7, 2019

Code Climate has analyzed commit b8629f2 and detected 0 issues on this pull request.

View more on Code Climate.

@commonality commonality deleted a comment Sep 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@codeclimate CodeClimate CodeClimate left review comments

Assignees

@gregswindle gregswindle

Labels
type: breaking change Modifications affect or replace the public API. ⇧ Bumps the MAJOR semver. type: dependency Update or pin a third-party library. ⇧ Bump the PATCH semver. type: refactor Restructuring source code without changing its external behavior. type: test Adding, correcting, or improving tests (coverage).
Projects
None yet
Milestone
Refactor modules with @sindresorhus.c...
1 participant
@gregswindle