-
Notifications
You must be signed in to change notification settings - Fork 8
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
CLI: Allow binaries to autoupdate #1540
Merged
Merged
Changes from 54 commits
Commits
Show all changes
58 commits
Select commit
Hold shift + click to select a range
e09d71d
autoupdate oclif
eemmiillyy 94b2df7
clean
eemmiillyy 434fa5f
use arn
eemmiillyy 6cb6f7b
types
eemmiillyy 1427ec2
only run on main
eemmiillyy 25fb9f3
use arn
eemmiillyy 9557679
id-token perm
eemmiillyy b32c17f
perm
eemmiillyy 240ff75
scoped permission
eemmiillyy d1d2fab
write
eemmiillyy 09fd500
var
eemmiillyy e14cbf2
chdir
eemmiillyy c6d297c
wait
eemmiillyy 3a31d21
use targets
eemmiillyy 7309fbe
continue
eemmiillyy 6569c47
add dep
eemmiillyy ca79754
wrong param
eemmiillyy 5702df0
acl
eemmiillyy 6838709
fix file name
eemmiillyy 619f4a0
target
eemmiillyy 72b406c
test
eemmiillyy 12c616b
fix
eemmiillyy dc90ebe
upload fix
eemmiillyy 326db88
pkg
eemmiillyy 44aa875
fix
eemmiillyy f09fbc0
promote
eemmiillyy d5b4655
test release gh
eemmiillyy e06d26a
host
eemmiillyy 6c4b528
extra slash
eemmiillyy a696988
workflow
eemmiillyy aabe943
deb
eemmiillyy 1773895
deb
eemmiillyy a73dfb7
deb
eemmiillyy e033dda
deb
eemmiillyy e2eb1d6
test
eemmiillyy ba9a21b
test
eemmiillyy df74102
test
eemmiillyy 091b329
separate steps
eemmiillyy 04b8a16
chdir
eemmiillyy b314368
run from dist folder
eemmiillyy 9294f43
skip windows tmp
eemmiillyy 61aa2d0
list files
eemmiillyy cf35914
cwd
eemmiillyy af77ad1
log error
eemmiillyy c60c242
test
eemmiillyy f7352bd
test
eemmiillyy eee5a5d
test
eemmiillyy da2760f
test
eemmiillyy 73d572e
windows
eemmiillyy c6b84d6
test gh
eemmiillyy 629599f
win path
eemmiillyy 9946471
clean
eemmiillyy 38440db
clean
eemmiillyy fa4cda3
pr changes
eemmiillyy 2e538c7
Merge branch 'main' into feat/cli-autoupdates
eemmiillyy 7224612
Create thick-bags-grab.md
SferaDev 12884e2
feat: setup autoupdates for homebrew formula (#1543)
eemmiillyy 9b2b2e9
node version
eemmiillyy File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains 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
This file contains 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
This file contains 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
This file contains 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
Oops, something went wrong.
Oops, something went wrong.
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.
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.
Not sure why we need this, also the version looks odd
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.
We need it because oclif throws an error if it is undefined: https://github.com/oclif/oclif/blob/main/src/tarballs/build.ts#L258
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.
What version should it be instead?
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.
18 probably