fix: replace js-yaml with yaml package #6469
Conversation
|
This pull request adds or modifies JavaScript ( |
mrstork
force-pushed
the
swap-yaml-package
branch
from
f243c4e to
d518ad9
Compare
|
|
mrstork
force-pushed
the
swap-yaml-package
branch
from
0f08935 to
1f10139
Compare
|
This pull request adds or modifies JavaScript ( |
|
This pull request adds or modifies JavaScript ( |
mrstork
force-pushed
the
swap-yaml-package
branch
from
907be6d to
3de19bf
Compare
|
This pull request adds or modifies JavaScript ( |
| @@ -1,6 +1,7 @@ | |||
| import figures from 'figures' | |||
There was a problem hiding this comment.
q: is there a policy by which you're deciding which files to convert to ts and which ones not to?
There was a problem hiding this comment.
The couple I changed were self contained, but when I started working with the other files there were subsequent changes that needed to be made. I simply drew a line in the sand, but ideally we can switch to all TS one day
|
This pull request adds or modifies JavaScript ( |
Summary
js-yamlwas last published 4 years agoyamlwas last published a month agoFor us to review and ship your PR efficiently, please perform the following steps:
we can discuss the changes and get feedback from everyone that should be involved. If you`re fixing a typo or
something that`s on fire 🔥 (e.g. incident related), you can skip this step.
your code follows our style guide and passes our tests.
A picture of a cute animal (not mandatory, but encouraged)