Skip to content

fix: missing exported types#1005

Merged
markov00 merged 3 commits intoelastic:masterfrom
markov00:2021_02_01-fix_missing_exported_types
Feb 2, 2021
Merged

fix: missing exported types#1005
markov00 merged 3 commits intoelastic:masterfrom
markov00:2021_02_01-fix_missing_exported_types

Conversation

@markov00
Copy link
Copy Markdown
Collaborator

@markov00 markov00 commented Feb 2, 2021

Summary

Export some missing types used in Kibana but not exported from the main module

@markov00 markov00 added the :build Build tools / dependencies label Feb 2, 2021
@markov00 markov00 requested a review from nickofthyme February 2, 2021 11:47
@codecov-io
Copy link
Copy Markdown

codecov-io commented Feb 2, 2021

Codecov Report

Merging #1005 (88a3ec2) into master (42372ae) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1005   +/-   ##
=======================================
  Coverage   72.48%   72.48%           
=======================================
  Files         353      353           
  Lines       11056    11056           
  Branches     2420     2420           
=======================================
  Hits         8014     8014           
  Misses       3027     3027           
  Partials       15       15           
Flag Coverage Δ
unittests 72.48% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...es/partition_chart/layout/types/viewmodel_types.ts 85.71% <ø> (ø)
...es/partition_chart/layout/utils/group_by_rollup.ts 87.87% <ø> (ø)
src/chart_types/heatmap/specs/heatmap.ts 66.66% <100.00%> (ø)
src/chart_types/heatmap/specs/index.ts 100.00% <100.00%> (ø)

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 1471ca9...88a3ec2. Read the comment docs.

Comment on lines +76 to +78
export const Heatmap: React.FunctionComponent<
Pick<HeatmapSpec, 'id' | 'data'> & Partial<Omit<HeatmapSpec, 'chartType' | 'specType' | 'id' | 'data'>>
> = getConnect()(
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Here I've tried to remove the type alias to avoid exporting an alias type.
We should do the same for all the other Components

Copy link
Copy Markdown
Collaborator

@nickofthyme nickofthyme left a comment

Choose a reason for hiding this comment

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

LGTM!

*/

export { Heatmap, HeatmapSpec } from './heatmap';
export * from './heatmap';
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

😃

@markov00 markov00 merged commit f6806de into elastic:master Feb 2, 2021
@markov00 markov00 deleted the 2021_02_01-fix_missing_exported_types branch February 2, 2021 21:59
github-actions Bot pushed a commit that referenced this pull request Feb 5, 2021
## [24.5.1](v24.5.0...v24.5.1) (2021-02-05)

### Bug Fixes

* missing exported types ([#1005](#1005)) ([f6806de](f6806de))
@markov00
Copy link
Copy Markdown
Collaborator Author

markov00 commented Feb 5, 2021

🎉 This PR is included in version 24.5.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@markov00 markov00 added the released Issue released publicly label Feb 5, 2021
AMoo-Miki pushed a commit to AMoo-Miki/OpenSearch-Dashboards that referenced this pull request Feb 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:build Build tools / dependencies released Issue released publicly

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants