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

fix: improve the exports of models #507

Merged
merged 2 commits into from
Nov 11, 2021
Merged

fix: improve the exports of models #507

merged 2 commits into from
Nov 11, 2021

Conversation

mortalYoung
Copy link
Collaborator

@mortalYoung mortalYoung commented Nov 11, 2021

简介

  • 优化 models 的导出
  • 支持 API 文档中解析 models 中的接口

Related Issues

Closed #503 , #496

@mortalYoung mortalYoung self-assigned this Nov 11, 2021
@mortalYoung mortalYoung added the bug Something isn't working label Nov 11, 2021
@codecov
Copy link

codecov bot commented Nov 11, 2021

Codecov Report

Merging #507 (7d5e52a) into main (00e8f7b) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #507   +/-   ##
=======================================
  Coverage   85.67%   85.67%           
=======================================
  Files         180      180           
  Lines        5290     5290           
  Branches     1185     1185           
=======================================
  Hits         4532     4532           
  Misses        751      751           
  Partials        7        7           
Impacted Files Coverage Δ
src/molecule.api.ts 100.00% <ø> (ø)
src/services/theme/colorThemeService.ts 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 00e8f7b...7d5e52a. Read the comment docs.

@wewoor wewoor linked an issue Nov 11, 2021 that may be closed by this pull request
@wewoor
Copy link
Collaborator

wewoor commented Nov 11, 2021

@mortalYoung
image

这个 Variables 也需要优化下,这两没必要在这儿导出

Copy link
Collaborator

@wewoor wewoor left a comment

Choose a reason for hiding this comment

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

+1

@wewoor wewoor merged commit 4b7ab03 into main Nov 11, 2021
@wewoor wewoor deleted the fix/interface branch November 11, 2021 07:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

API document lost the content of IFolderTree interface Update the default Molecule API
2 participants