Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Massive Refactor (part II) 🍻 - WIP#85

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
alexcanessa merged 19 commits intorelease-0.10.0fromsecond-refactor
Sep 29, 2017

Conversation

@alexcanessa
Copy link
Member

@alexcanessaalexcanessa commentedSep 24, 2017
edited
Loading

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 thegren command and both generate yml files in thedocs folder, to feedjekyll.

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

README.md

resolves#78

Updated theREADME.md file to reflectOptions and Examples and removed it from thedocs where now the homepage is just the navigation (don't need a duplicate for that).

Testing -currently WIP

resolves#37

Removednodeunit and now using the bundlemocha,chai,istanbul andnyc to run test and coverage.

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

@alexcanessaalexcanessaforce-pushed thesecond-refactor branch 3 times, most recently from7450e33 tod2e4adaCompareSeptember 25, 2017 10:22
@codecov-io
Copy link

codecov-io commentedSep 25, 2017
edited
Loading

Codecov Report

❗ No coverage uploaded for pull request base (master@30500d6).Click here to learn what that means.
The diff coverage is75.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 FilesCoverage Δ
lib/src/GitHubInfo.js88.88% <100%> (ø)
lib/_options.js100% <100%> (ø)
lib/src/_template.js100% <100%> (ø)
lib/src/_utils.js100% <100%> (ø)
lib/src/Program.js77.96% <72.22%> (ø)
lib/src/Gren.js75.34% <73.27%> (ø)

Continue to review full report at Codecov.

Legend -Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing data
Powered byCodecov. Last update30500d6...a1a8d37. Read thecomment docs.

@alexcanessaalexcanessaforce-pushed thesecond-refactor branch 4 times, most recently from194605d to9777c38CompareSeptember 25, 2017 16:53
@alexcanessaalexcanessaforce-pushed thesecond-refactor branch 2 times, most recently fromc9655cf to7d7e99dCompareSeptember 29, 2017 17:28
@alexcanessaalexcanessa changed the base branch frommaster torelease-0.10.0September 29, 2017 20:10
@alexcanessaalexcanessa merged commitce3cb3b intorelease-0.10.0Sep 29, 2017
@alexcanessaalexcanessa deleted the second-refactor branchSeptember 29, 2017 20:11
alexcanessa added a commit that referenced this pull requestSep 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
@alexcanessaalexcanessa mentioned this pull requestOct 8, 2017
alexcanessa added a commit that referenced this pull requestOct 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
This was referencedOct 8, 2017
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@alexcanessa@codecov-io

[8]ページ先頭

©2009-2025 Movatter.jp