Skip to content

Migrate kumascript jsondata to content repo - #8019

Merged
sideshowbarker merged 4 commits into
mdn:mainfrom
peterbe:migrate-kumascript-jsondata-to-content-repo
Aug 21, 2021
Merged

sideshowbarker merged 4 commits into
mdn:mainfrom
peterbe:migrate-kumascript-jsondata-to-content-repo

Conversation

@peterbe

@peterbe peterbe commented Aug 17, 2021

Copy link
Copy Markdown
Contributor

This is the company of mdn/yari#4081

@peterbe
peterbe requested review from a team and fiji-flo and removed request for a team and fiji-flo August 17, 2021 17:07
@teoli2003

teoli2003 commented Aug 20, 2021 •

Copy link
Copy Markdown
Contributor

Should we merge this so we can start modifying these files here? Even if we won't use them before Yari's PR lands?

@peterbe

peterbe commented Aug 20, 2021

Copy link
Copy Markdown
Contributor Author

I think so.
The next action item is actually:

  1. A refresh of move kumascript .json files to content (yari) yari#4081 (to prove the tests pass)
  2. @escattone to review move kumascript .json files to content (yari) yari#4081

@escattone If you think we can get around to the yari side of this within days, I think we can go ahead with this here PR. In fact, then the content folks can start actually sending PRs to these even if it's a bit delayed till it actually appears in production builds.

@escattone

Copy link
Copy Markdown
Contributor

@peterbe Yes, sounds good to me!

@peterbe

peterbe commented Aug 20, 2021

Copy link
Copy Markdown
Contributor Author

@peterbe Yes, sounds good to me!

So, mergy merge?

@sideshowbarker
sideshowbarker merged commit 9d71565 into mdn:main Aug 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants