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

Add support for source-locators #4

Merged
merged 4 commits into from
Mar 14, 2017
Merged

Add support for source-locators #4

merged 4 commits into from
Mar 14, 2017

Conversation

nknapp
Copy link
Member

@nknapp nknapp commented Mar 13, 2017

  • Add an options for activating source-locators

If activated, the following is done:

  • Use "handlebars-source-locators" to add source-locator tags
    to the output
  • Post-process the output to insert actual filenames into the tags

@nknapp
Add support for source-locators
ff0adfa 
- Add an options for activating source-locators

If activated, the following is done:

- Use "handlebars-source-locators" to add source-locator tags
  to the output
- Post-process the output to insert actual filenames into the tags
@coveralls
Copy link

coveralls commented Mar 13, 2017 •
edited
Loading

Coverage Status

Coverage decreased (-3.2%) to 92.857% when pulling b1762db on source-locators into 463251d on master.

@coveralls
Copy link

coveralls commented Mar 13, 2017 •
edited
Loading

Coverage Status

Coverage decreased (-3.2%) to 92.857% when pulling 69a4134 on source-locators into 463251d on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.7%) to 96.774% when pulling dcf145d on source-locators into 463251d on master.

1 similar comment
@coveralls
Copy link

coveralls commented Mar 14, 2017 •
edited
Loading

Coverage Status

Coverage increased (+0.7%) to 96.774% when pulling dcf145d on source-locators into 463251d on master.

@coveralls
Copy link

coveralls commented Mar 14, 2017 •
edited
Loading

Coverage Status

Coverage increased (+0.7%) to 96.774% when pulling a462915 on source-locators into 463251d on master.

@coveralls
Copy link

coveralls commented Mar 14, 2017 •
edited
Loading

Coverage Status

Coverage increased (+0.7%) to 96.774% when pulling bf17721 on source-locators into 463251d on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.7%) to 96.774% when pulling 5b61da4 on source-locators into 463251d on master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage increased (+0.7%) to 96.774% when pulling 5b61da4 on source-locators into 463251d on master.

@coveralls
Copy link

coveralls commented Mar 14, 2017 •
edited
Loading

Coverage Status

Coverage increased (+0.7%) to 96.774% when pulling be39a37 on source-locators into 463251d on master.

@nknapp
BREAKING: Support only node 6 and 7. Updated CI config
c639b39 
The introduction of source-locators breaks compatibility with
node-versions below 6, because the package 'handlebars-source-locators'
uses language features that are only available in node 6.

In addition, this commit introduces minor changes in the Travis-
configuration:

- "standard" is now a dev-dependency to enable controlled updates
  through Greenkeeper
- "thougth -d up-to-date" is run in Travis-CI to make sure the
  documentation is up-to-date
- Examples are not executed with a custom `console.log`-function
  that uses "json-stable-stringify" to create a stable output. This
  has been done because "`"thought up-to-date" compares the checked-in
  documentation to the documentation that would be generated right now.
  This requires the output of examples to be stable.
- Reduce load on traffic by executing Thought and Standard in their own
  build job (and not for each node version)
- Add caching-options to .travis.yml
- Run standard as precommit hook rathern than pre-push, because
  the check then always refers to the active branch.
- Run "standard --fix" on precommit rather than just "standard", because
  it is require most of the time, when "standard" fails.
@coveralls
Copy link

Coverage Status

Coverage increased (+0.7%) to 96.774% when pulling 54739d5 on source-locators into 463251d on master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage increased (+0.7%) to 96.774% when pulling 54739d5 on source-locators into 463251d on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.7%) to 96.774% when pulling c639b39 on source-locators into 463251d on master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage increased (+0.7%) to 96.774% when pulling c639b39 on source-locators into 463251d on master.

@nknapp
Disable "thought up-to-date" in Travis
aa006af 
The command sometimes errors for no obvious reason.
@coveralls
Copy link

Coverage Status

Coverage increased (+0.7%) to 96.774% when pulling aa006af on source-locators into 463251d on master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage increased (+0.7%) to 96.774% when pulling aa006af on source-locators into 463251d on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+3.9%) to 100.0% when pulling cc0bf3e on source-locators into 463251d on master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage increased (+3.9%) to 100.0% when pulling cc0bf3e on source-locators into 463251d on master.

@nknapp nknapp merged commit 3a76089 into master Mar 14, 2017
@nknapp nknapp deleted the source-locators branch March 14, 2017 22:42
Sign up for free to 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.

None yet
2 participants
@nknapp @coveralls