-
Notifications
You must be signed in to change notification settings - Fork 189
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
Use the latest satellite version #2664
Conversation
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. I was going to say you missed the autodeployment server, but it already does this
"@senecacdot/satellite": "^1.x", |
Just rebased.
Did Renovate miss this when it was pinning dependencies? |
Not sure, I wonder if it did. |
1 question, |
Yeah, It depends on which one gets merged first. It'll be best that you also use |
Yup, I'll change it to 1.x for less work. |
Renovate doesn't like autodeployment, no dependencies were pinned there. |
This changes the Satellite version from
1.17
to1.x
. We manage Satellite ourselves so it's safe to not pin it and let ppm install the latest minor/patch if any.Checklist