-
Notifications
You must be signed in to change notification settings - Fork 15
Added support to delete codespaces just like repositories #37
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
Merged
Merged
Changes from all commits
Commits
Show all changes
47 commits
Select commit
Hold shift + click to select a range
00ed45b
feat: added support to delete codespaces
AnishDe12020 6197f45
update: bump version
adrianmg 21b41b8
fix(codespace): make user re-auth if unable to fetch codespaces
AnishDe12020 55803a0
refactor: use commander.js
AnishDe12020 bcfce4a
refactor: remove commander.js
AnishDe12020 a8eae41
test: added codespace scope to tests
AnishDe12020 587bb88
docs: update readme to show usage of codespaces command and update fi…
AnishDe12020 8ed64c2
docs(readme): fix indentation in file tree
AnishDe12020 5acd3ec
fix: print help if unknown command is passed
AnishDe12020 865b2b7
Merge branch 'feat/delete-codespaces' of github.com:AnishDe12020/gith…
AnishDe12020 51ecf43
refactor: use a utility function for repo/codespace label
AnishDe12020 10b0adf
update: add anish to README
adrianmg 5f582ae
fix: stop printing help message if no repos selected
AnishDe12020 a0e7a7d
update: refactor arrow function for consistency
adrianmg ec7014f
update: more descriptive name for labels func
adrianmg 1397859
update: package description
adrianmg 602026f
improve: help command formatting
adrianmg 31de3a6
update: README with more todos
adrianmg 36ef81e
Merge branch 'feat/delete-codespaces' of github.com:AnishDe12020/gith…
adrianmg efc1a51
fix: missing string
adrianmg a5e8cda
fix: correct print message on canceling deletion of codespaces
adrianmg bbb23fd
update: const variables from package
adrianmg 6397aae
update: make help command the default one
adrianmg 79647eb
Bump clipboardy from 2.3.0 to 3.0.0
dependabot[bot] e794311
update: package description
adrianmg 0f129b0
Bump ora from 5.4.1 to 6.1.2
dependabot[bot] d271fa9
Migration from CommonJS to ES Modules
adrianmg e341924
update: test to ES Modules
adrianmg 12aac3a
Bump mocha from 9.2.2 to 10.0.0
adrianmg 7d7bb93
Bump @octokit/auth-oauth-device from 3.1.2 to 4.0.2
adrianmg 5db9794
add: hint how to select
adrianmg 10cffb0
update: less obvious hint to select with space?
adrianmg 9ef7d97
add: backup commands alias for repos
adrianmg 809d683
update: document available commands
adrianmg f90b2c5
fix: importing assert
adrianmg 5750a92
fix: tests workflow bump version 16
adrianmg 02c9b8e
update: workflow can be dispatched manually
adrianmg 840dd63
fix: add back npm install
adrianmg 9822d5b
update: minor refactor
adrianmg f6c7e92
update: permissions for release drafter action
adrianmg 1aaf5b3
test if this fixes it
adrianmg b3f530e
fix
adrianmg 3f09c5d
remove: release draft is not needed anymore. BYE
adrianmg 4058d55
merge: pr #39
jdvr 44b7954
update: directory is recursive and takes unix into account
adrianmg 35e46fe
update: homedir() directly imported from 'os'
adrianmg 45a252d
update: package description since now you can do more than repos!
adrianmg 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 was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,12 +1,12 @@ | ||
| name: Tests | ||
| on: pull_request | ||
| on: [pull_request, workflow_dispatch] | ||
| jobs: | ||
| test: | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - uses: actions/checkout@v2 | ||
| - uses: actions/setup-node@v2 | ||
| - uses: actions/checkout@v3 | ||
| - uses: actions/setup-node@v3 | ||
| with: | ||
| node-version: '10.x' | ||
| node-version: 16 | ||
| - run: npm install | ||
| - run: npm test |
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
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
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.
This is right in order to avoid breaking changes in how the tool is executed. However, I think this could be improved. For example, if anyone executes something like this:
The script will execute the
reposCommand. Maybe we could change this behavior to print the help: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.
Ideally, it should be tested too...
Uh oh!
There was an error while loading. Please reload this page.
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.
That makes sense, I have added the check (5acd3ec)
For the label part, I have made this utility function -
(51ecf43)
The code is much more cleaner now, thanks for the feedback :)