Skip to content

fix(save): implement "save" method for Node.js API#17

Merged
gregswindle merged 6 commits intomasterfrom
GH-16-fix-grc-save
Apr 4, 2018
Merged

fix(save): implement "save" method for Node.js API#17
gregswindle merged 6 commits intomasterfrom
GH-16-fix-grc-save

Conversation

@gregswindle
Copy link
Copy Markdown
Owner

@gregswindle gregswindle commented Apr 4, 2018

Description of change

  1. Add a save method to baseResourceConveter, which invokes
    fileExporter.save. 😒
  2. Update API section of the README.
  3. Enumerate README sections.
  4. Add .npmignore and .npmrc files.
  5. Update ISSUE_TEMPLATE.md, defect-report.md, and automated query parameters for "Report a defect" and "Request a feature" buttons.
  6. Ensure all tests pass with 100% coverage.
  7. Move the CLI's "help"/"usage" copy to an i18n directory.
  8. Attempt to generate a CHANGELOG with semantic-release 🤞 .
  9. Update outdated dependencies.

Associated issue(s)

Resolves #16
Resolves #15

Acceptance criteria/task completion

  • The acceptance criteria for all associated issues have been completed, tested, and validated.

PR check-list

✅ Please review and check the appropriate items.

1. Code standards compliance

JavaScript Style Guide

  • ESLint passes.

2. Code quality Quality Gate

  • The quality gateways pass with an "A" grade.
Measure Scores
Complexity Complexity
Cognitive complexity
Complexity per Class
Complexity per file
Complexity per Function
Duplications Duplications
Issues Issues
Maintainability Code smells
Maintainability
Technical debt
Reliability Reliability
Security Security

3. Test coverage Coverage Status

  • The source code is 100% covered with passing specs.

** These tasks are not required to open a PR, and may be addresses while the PR is open.**

Added a #save method to baseResourceConveter, which invokes
fileExporter.save. 😒

GH-16
Enumerate all sections.

GH-16
@gregswindle gregswindle added priority: high Product functions, but there's risk of abandonment; address ASAP type: docs Changes to material that provides information. status: accepted Defect confirmed or feature endorsed for work type: fix Defect repairs. ⇧ Bumps the PATCH semver. status: review needed type: build Changes that affect the build system or external dependencies. type: test labels Apr 4, 2018
@gregswindle gregswindle added this to the Fix CLI (v1.0.1) milestone Apr 4, 2018
@gregswindle gregswindle self-assigned this Apr 4, 2018
@gregswindle
Copy link
Copy Markdown
Owner Author

LGTM! 👍

@gregswindle gregswindle merged commit 7e22c30 into master Apr 4, 2018
@gregswindle gregswindle deleted the GH-16-fix-grc-save branch April 4, 2018 07:59
gregswindle added a commit that referenced this pull request Apr 4, 2018
* fix(save): ensure module's save method works

   Add `save` method to baseResourceConveter, which invokes `fileExporter.save`.

* docs(readme): update API section

   Enumerate all sections.

* style(manifest): format

* docs(readme): fix typo

   Replace `octokit` reference with `grc`.

* fix(save): add save method to module

GH-16, GH-17, GH-18
gregswindle added a commit that referenced this pull request Apr 4, 2018
* fix(save): ensure module's save method functions

Added a #save method to baseResourceConveter, which invokes
fileExporter.save. 😒

GH-16

* docs(readme): update API section

Enumerate all sections.

GH-16

* style(manifest): format

GH-16

* docs(readme): fix typo

Replace octokit ➡️ grc

GH-16,GH-17

* fix(save): add save method to module

GH-18

* build(changelog): add standard-version

Use standard-version to generate a CHANGELOG, since semantic-release
does not appear to have this feature.

GH-18

* docs(changelog): add v1.0.0 to log

GH-18
@gregswindle
Copy link
Copy Markdown
Owner Author

🎉 This PR is included in version 1.0.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

priority: high Product functions, but there's risk of abandonment; address ASAP status: accepted Defect confirmed or feature endorsed for work status: review needed type: build Changes that affect the build system or external dependencies. type: docs Changes to material that provides information. type: fix Defect repairs. ⇧ Bumps the PATCH semver. type: test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant