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

Initial version #1

Merged
merged 7 commits into from
Dec 18, 2018
Merged

Initial version #1

merged 7 commits into from
Dec 18, 2018

Conversation

thisandagain
Copy link
Contributor

@thisandagain thisandagain commented Dec 17, 2018

Overview

Sets up the initial version of scratch-analysis which replaces functionality from version 1.0.2 of scratch-parser while also adding identical support for SB3 files.

Context

This work is required to update the Scratch search crawler to support SB3 files prior to launch.

Test Coverage

image

lib/index.js Outdated Show resolved Hide resolved
analyze.js Outdated Show resolved Hide resolved
lib/index.js Outdated Show resolved Hide resolved
lib/index.js Outdated Show resolved Hide resolved
lib/sb2.js Outdated Show resolved Hide resolved
lib/sb3.js Outdated Show resolved Hide resolved
@thisandagain
Copy link
Contributor Author

thisandagain commented Dec 18, 2018

Thanks @kchadha. I addressed your comments with the exception of one (which I've opened an issue for on scratch-parser). Nice side effect of removing the default switch statement 😄:
image

Copy link

@kchadha kchadha left a comment

Choose a reason for hiding this comment

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

LGTM!

@kchadha kchadha assigned thisandagain and unassigned kchadha Dec 18, 2018
@thisandagain thisandagain merged commit 50d9f83 into scratchfoundation:master Dec 18, 2018
Copy link

github-actions bot commented Nov 7, 2024

🎉 This PR is included in version 2.1.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants