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.

Sign up for GitHub

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

Jump to bottom

Use js-yaml (instead of yaml.js) for YAML parsing #166

Merged
merged 4 commits into from
Feb 10, 2024
Merged

Use js-yaml (instead of yaml.js) for YAML parsing #166

merged 4 commits into from
Feb 10, 2024

Conversation

generalmimon
Copy link
Member

GreyCat
GreyCat reviewed Feb 5, 2024
View reviewed changes
vendor.yaml Show resolved Hide resolved
@generalmimon generalmimon mentioned this pull request Feb 9, 2024
Open
@generalmimon generalmimon mentioned this pull request Feb 10, 2024
Open
@generalmimon
Copy link
Member Author

@GreyCat Thanks for the review!

@generalmimon generalmimon merged commit 5de330d into master Feb 10, 2024
1 check passed
@generalmimon generalmimon deleted the switch-to-js-yaml branch February 10, 2024 23:25
generalmimon added a commit to kaitai-io/kaitai-io.github.io that referenced this pull request Feb 22, 2024
@generalmimon
Add post about the latest Web IDE improvements
75c77c1 
Related Web IDE pull requests with the changes:

* kaitai-io/kaitai_struct_webide#170
* kaitai-io/kaitai_struct_webide#169
* kaitai-io/kaitai_struct_webide#166
* kaitai-io/kaitai_struct_webide#162
* kaitai-io/kaitai_struct_webide#163
* kaitai-io/kaitai_struct_webide#161
@generalmimon generalmimon mentioned this pull request Feb 22, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@GreyCat GreyCat GreyCat approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@generalmimon @GreyCat