You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Adds existing sync target that generates from existing lockfile entries without upload any assets. This is useful for CI linux runners to generate a production-ready asset file without uploading.
Also if any asset's hash is not found in the lockfile the command fails with an error
Hey. I'm not completely sold on adding this target-I think it's a bit strange to have a separate one from Cloud that uses the lockfile, which only Cloud uses.
Can you let me know what you gain from not checking in your generated asset files? This is the standard workflow and also makes your code typecheckable without needing an extra step.
Can you let me know what you gain from not checking in your generated asset files? This is the standard workflow and also makes your code typecheckable without needing an extra step.
My use for this may be a bit specific. Assets are git-ignored and whenever testing locally we use the studio generated asset file so type safety always works. Once changes are merged, there's a pipeline that generates the asset file with the public rbx id's, then a rbxl is built and uploaded to a test place.
Before merging there's a workflow that uses the existing target to test for any new assets that haven't been cloud synced yet.
Hey. I'm not completely sold on adding this target-I think it's a bit strange to have a separate one from Cloud that uses the lockfile, which only Cloud uses.
If it makes more sense, I can make it an option for the cloud target instead
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
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.
Adds
existingsync target that generates from existing lockfile entries without upload any assets. This is useful for CI linux runners to generate a production-ready asset file without uploading.Also if any asset's hash is not found in the lockfile the command fails with an error
usage:
asphalt sync existing