Skip to content

docs: fix the example in README.md #57

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

Closed
wants to merge 2 commits into from
Closed

docs: fix the example in README.md #57

wants to merge 2 commits into from

Conversation

stahlbauer
Copy link
Contributor

Minor adjustment of the example code in README.md to make it run without an exception.

The second parameter of parse determines whether the given buffer contains a sprite or a project.
Fixed the example code in the README.md
@thisandagain
Copy link
Contributor

Thank you!

@thisandagain
Copy link
Contributor

@stahlbauer This looks good with the exception of the commit formatting. Do you mind installing and using commitizen so we can follow semantic release?

https://github.com/LLK/scratch-parser#committing

@thisandagain thisandagain added this to the November 2019 milestone Oct 7, 2019
@BryceLTaylor BryceLTaylor modified the milestones: November 2019, Overdue Jul 13, 2020
@seheon99
Copy link

seheon99 commented Mar 14, 2023

Please merge this request.. 🥺

cwillisf
cwillisf previously approved these changes Dec 15, 2023
Copy link
Contributor

@cwillisf cwillisf left a comment

Choose a reason for hiding this comment

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

Thanks for this update; sorry for the delay!

Please format future commit messages according to this standard: https://github.com/bcoe/conventional-changelog-standard/blob/master/convention.md

@cwillisf cwillisf changed the title Fix of the example in README.md docs: fix the example in README.md Dec 15, 2023
@cwillisf
Copy link
Contributor

Merged by hand: 572edf9

@cwillisf cwillisf closed this Dec 15, 2023
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.

5 participants