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

Feat/addons docs tsup #18788

Closed
wants to merge 5 commits into from
Closed

Feat/addons docs tsup #18788

wants to merge 5 commits into from

Conversation

Saunved
Copy link
Contributor

@Saunved Saunved commented Jul 25, 2022

Issue: #18732

What I did

Followed the instructions on the issue mentioned above

How to test

  • Is this testable with Jest or Chromatic screenshots? Yes, with CI
  • Does this need a new example in the kitchen sink apps? No
  • Does this need an update to the documentation? No

If your answer is yes to any of these, please make sure to include it in your PR.

@Saunved Saunved added the maintenance User-facing maintenance tasks label Jul 25, 2022
@Saunved Saunved requested a review from ndelangen July 25, 2022 19:36
@Saunved
Copy link
Contributor Author

Saunved commented Jul 25, 2022

@ndelangen

Based on this deployment failure: https://app.netlify.com/sites/storybookjs/deploys/62def0a92e5b730008aa0ea6
Looks like the docs addon needs to export angular. I can't spot it in the old build, so not sure how it was being accessed.

I am not sure how to export angular from this repository with the new build system. Do I add it to exports? Would it be the same for ember and web-components?

# Conflicts:
#	code/addons/docs/package.json
@ndelangen
Copy link
Member

@Saunved Seems this addon is a bit more involved than the others.. Let's revisit this one last.

@Saunved
Copy link
Contributor Author

Saunved commented Aug 1, 2022

Yes, it seems a bit complicated. I'll push the remaining ones this week for review!

},
"dependencies": {
"@babel/plugin-transform-react-jsx": "^7.12.12",
"@babel/preset-env": "^7.12.11",
"@digitak/esrun": "^3.2.2",
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
"@digitak/esrun": "^3.2.2",

@@ -6566,6 +6566,7 @@ __metadata:
"@babel/core": ^7.12.10
"@babel/plugin-transform-react-jsx": ^7.12.12
"@babel/preset-env": ^7.12.11
"@digitak/esrun": ^3.2.2
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
"@digitak/esrun": ^3.2.2

@ndelangen
Copy link
Member

Hello @Saunved sorry this PR was somewhat neglected.. it turned out this package was quite complex, and so I took it over at some point for forgot to inform you.

Sorry about that. I hope you don't mind me closing this PR now.

Thank you for your time, energy and effort put into this (and the others) PR. It's greatly appreciated!

@ndelangen ndelangen closed this Nov 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance User-facing maintenance tasks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants