-
Notifications
You must be signed in to change notification settings - Fork 1
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
feat: tasks and lockfiles #28
Conversation
2f4bbbf
to
364739f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good so far
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One small suggestion, otherwise lgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Love the new lock file :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
huge work!
Describe your change
First crack at tasks.
Motivation and context
#10
Migration notes
ghjk
executable now looks for or generates a.ghjk
directory next toghjk.ts
.$GHJK_CONFIG
has been renamed to$GHJKFILE
$GHJK_DIR
meaning has been changed to point at the discovered.ghjk
dir as opposed to the ghjk share dir.ghjk.ts
serialization might be skipped if the it's been determined that no important inputs have changed.InstallConfig
.version
is now checked against the versions returned by the port'slistAll
implementation.InstallConfig
.version
is now cached in the lockfile. For now, manually remove the lockfile to re-resolve everything.install
anddownload
steps on a port are skipped if they'd previously successfully completed on the sameInstallConfig
.Checklist