docs: document alternative approach for local development #8
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
floehopper
approved these changes
Apr 16, 2025
There was a problem hiding this comment.
This looks good to me 👍
A couple of minor subjective points:
- It might be better to use
ScratchFoundationas the org name in theRaspberryPiFoundation-scratch-gui-4.1.0-beta.1.tgzexample filename. - I think it would make more sense to include the new section after the "Getting another repo to point to this code (using
npm link)" section, i.e. just after the "Oh no! It didn't work!" section and just before the "Testing" section. What do you think?
Whilst recently attempting to test some changes to the webpack config we struggled to get `npm link` to work. We ended up using an alternative approach using `npm pack` which generates a tarball of the npm package which you can then `npm install` into your other project. In order to help future developers who find themselves in a similar situation I thought it was worth documenting this alternative approach.
sebjacobs
force-pushed
the
seb-testing-locally-instructions
branch
from
cf638ef to
276c402
Compare
sebjacobs
pushed a commit
that referenced
this pull request
Apr 24, 2025
sebjacobs
pushed a commit
that referenced
this pull request
Apr 30, 2025
sebjacobs
pushed a commit
that referenced
this pull request
Apr 30, 2025
sebjacobs
pushed a commit
that referenced
this pull request
Apr 30, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Whilst recently attempting to test some changes to the webpack config we struggled to get
npm linkto work.We ended up using an alternative approach using
npm packwhich generates a tarball of the npm package which you can thennpm installinto your other project.In order to help future developers who find themselves in a similar situation I thought it was worth documenting this alternative approach.