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

docs(data-point): add @class tags #431

Merged
merged 2 commits into from
Sep 19, 2019

Conversation

csarnataro
Copy link
Collaborator

@csarnataro csarnataro commented Sep 18, 2019

What:
Replaced jsdoc @alias tag with more meaningful @class and @classdesc tags
Apparently the reason why @alias was used is that it allows to create automatic links between jsdoc pages. But after doing some experiments, it just turned out that also @class tags generate automatic links between pages.
See here for example:
https://csarnataro.github.io/data-point/jsdoc/Reducer.html#resolveReducer
If you click on Accumulator, it will take you to the Accumulator documentation page https://csarnataro.github.io/data-point/jsdoc/Accumulator.html

Why:
To use the more semantically appropriate tag.

How:
Just replacing @alias with @class and adding @classdesc where applicable.

Checklist:

  • Has Breaking changes N/A
  • Documentation
  • Tests N/A
  • Ready to be merged
  • Added username to all-contributors list

Closes #422

Please see https://csarnataro.github.io/data-point/jsdoc/ or https://csarnataro.github.io/data-point/jsdoc/Reducer.html#resolveReducer to check if this is the intended behaviour of internal links

Christian Sarnataro added 2 commits September 18, 2019 23:22
Replaced @alias tags with @Class and @classdesc tags. Added DataPoint temporary favicon.

Closes issue no. 422
@codecov
Copy link

codecov bot commented Sep 18, 2019

Codecov Report

Merging #431 into canary will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           canary   #431   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          29     29           
  Lines         482    482           
  Branches       74     74           
=====================================
  Hits          482    482
Impacted Files Coverage Δ
...es/data-point/src/reducer-helpers/assign/assign.js 100% <ø> (ø) ⬆️
packages/data-point/src/tracing/Tracer.js 100% <ø> (ø) ⬆️
packages/data-point/src/reducer-helpers/map/map.js 100% <ø> (ø) ⬆️
packages/data-point/src/ReducerObject.js 100% <ø> (ø) ⬆️
packages/data-point/src/DataPoint.js 100% <ø> (ø) ⬆️
packages/data-point/src/ReducerEntity.js 100% <ø> (ø) ⬆️
packages/data-point/src/Accumulator.js 100% <ø> (ø) ⬆️
packages/data-point/src/Reducer.js 100% <ø> (ø) ⬆️
packages/data-point/src/tracing/Span.js 100% <ø> (ø) ⬆️
...ata-point/src/reducer-helpers/constant/constant.js 100% <ø> (ø) ⬆️
... and 7 more

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 c85c526...6234199. Read the comment docs.

@csarnataro csarnataro changed the title Docs/jsdoc add class tags docs(data-point): add @class tags Sep 18, 2019
Copy link
Collaborator

@acatl acatl left a comment

Choose a reason for hiding this comment

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

thanks!

packages/data-point/src/Accumulator.js Show resolved Hide resolved
@@ -5,10 +5,11 @@ function normalizeName(name) {
}

/**
* @alias Reducer
* @class
* @classdesc This class is never instantiated directly, it is used to create
Copy link
Collaborator

Choose a reason for hiding this comment

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

good comment! thanks!

@acatl acatl merged commit e7b0b3f into ViacomInc:canary Sep 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants