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.

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

Massive Refactor (part II) 🍻 - WIP #85

Merged
merged 19 commits into from
Sep 29, 2017
Merged

Conversation

alexcanessa
Copy link
Member

@alexcanessa alexcanessa commented Sep 24, 2017

This is a massive PR, indeed

Docs

Lot of work of this PR has been done to make the documentation more readable.

Options and Examples

contributes to #66 and #80 (already started in #82).

Options and Examples are now implemented in the gren command and both generate yml files in the docs folder, to feed jekyll.

The documentation now implements the options and the examples shown on the terminal.

README.md

resolves #78

Updated the README.md file to reflect Options and Examples and removed it from the docs where now the homepage is just the navigation (don't need a duplicate for that).

Testing - currently WIP

resolves #37

Removed nodeunit and now using the bundle mocha, chai, istanbul and nyc to run test and coverage.

  • The coverage badge is now in the README.md file.
  • Codecov report on PRs.

@alexcanessa alexcanessa force-pushed the second-refactor branch 3 times, most recently from 7450e33 to d2e4ada Compare September 25, 2017 10:22
@codecov-io
Copy link

codecov-io commented Sep 25, 2017

Codecov Report

❗ No coverage uploaded for pull request base (master@30500d6). Click here to learn what that means.
The diff coverage is 75.44%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master      #85   +/-   ##
=========================================
  Coverage          ?   79.27%           
=========================================
  Files             ?        7           
  Lines             ?      333           
  Branches          ?        0           
=========================================
  Hits              ?      264           
  Misses            ?       69           
  Partials          ?        0
Impacted Files Coverage Δ
lib/src/GitHubInfo.js 88.88% <100%> (ø)
lib/_options.js 100% <100%> (ø)
lib/src/_template.js 100% <100%> (ø)
lib/src/_utils.js 100% <100%> (ø)
lib/src/Program.js 77.96% <72.22%> (ø)
lib/src/Gren.js 75.34% <73.27%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 30500d6...a1a8d37. Read the comment docs.

@alexcanessa alexcanessa force-pushed the second-refactor branch 4 times, most recently from 194605d to 9777c38 Compare September 25, 2017 16:53
@alexcanessa alexcanessa force-pushed the second-refactor branch 2 times, most recently from c9655cf to 7d7e99d Compare September 29, 2017 17:28
@alexcanessa alexcanessa changed the base branch from master to release-0.10.0 September 29, 2017 20:10
@alexcanessa alexcanessa merged commit ce3cb3b into release-0.10.0 Sep 29, 2017
@alexcanessa alexcanessa deleted the second-refactor branch September 29, 2017 20:11
alexcanessa added a commit that referenced this pull request Sep 29, 2017
* Add all the defaults to Release.js

* First change for README.md file

* Create Gren task and add functionality for the changelog command

* Update node version in travis

* Change the task for the Gren

* Update options description

* Use native Object.assign for the program options to avoid deep merging

* Prettify the utils task logger

* Accept * as all tags

* Get github env informations from Program.js

* Update the generated documentation

* Change the name in the docs

* Update documentation

* Add code coverage with mocha/istanbul

* Update organisation name in the package.json

* Get the token from process.env rather than command

* Update documentation

* Create tests

* Add npm downloads badge
@alexcanessa alexcanessa mentioned this pull request Oct 8, 2017
alexcanessa added a commit that referenced this pull request Oct 8, 2017
* Add all the defaults to Release.js

* First change for README.md file

* Create Gren task and add functionality for the changelog command

* Update node version in travis

* Change the task for the Gren

* Update options description

* Use native Object.assign for the program options to avoid deep merging

* Prettify the utils task logger

* Accept * as all tags

* Get github env informations from Program.js

* Update the generated documentation

* Change the name in the docs

* Update documentation

* Add code coverage with mocha/istanbul

* Update organisation name in the package.json

* Get the token from process.env rather than command

* Update documentation

* Create tests

* Add npm downloads badge
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants