Skip to content
This repository has been archived by the owner on Dec 31, 2021. It is now read-only.

Do not support bucket without nested bucket folder #31

Closed
Ash258 opened this issue Jul 6, 2019 · 1 comment
Closed

Do not support bucket without nested bucket folder #31

Ash258 opened this issue Jul 6, 2019 · 1 comment
Assignees
Labels
enhancement New feature or request high-priority
Milestone

Comments

@Ash258
Copy link
Owner

Ash258 commented Jul 6, 2019

This will save me lots of checks and my sanity.

If bucket do not have nested bucket folder fail and create new issue about this

@Ash258 Ash258 added enhancement New feature or request high-priority labels Jul 6, 2019
@Ash258 Ash258 self-assigned this Jul 6, 2019
@Ash258 Ash258 added this to the 1.0.0 Release milestone Jul 6, 2019
@Ash258
Copy link
Owner Author

Ash258 commented Jul 9, 2019

Entrypoint.ps1

if (-not (Test-Path $BUCKET_ROOT\bucket)) {
    $js = Invoke-GithubRequest ".../issues/" | Select-Object -ExpandProperty Content | ConvertFrom-Json
	$js | Where-Object { $_.Author -eq 'Me' -and $_.Title -eq 'Use nested bucket' }
	if (-not $js) { New-Issue }
}

Ash258 added a commit that referenced this issue Jul 20, 2019
@Ash258 Ash258 closed this as completed in 5351a40 Jul 20, 2019
Ash258 added a commit that referenced this issue Aug 3, 2019
* Only nested bucket

#31

* Initial support for aria2 downloading

* Better message for download action

#15

* Wording

* Simplify

* Just test for new issue

* idiot

* Empty arrays

* ...

* null check

* Existence check

* Existence check

* Log

* Now?

* Now?

* List

* Count

* Count

* count

* Done

* Move nested bucket folder test into function

- Closes #31

* Simplify Hash test

#10

* Status

* Correct gci call

* Strict file name for gci

* idiot ... 🤦‍♀️🤦‍♀️🤦‍♀️

* Test-hash is now good

* Log pure event from PR

* test

* Readme

* Handle possible version difference

#37

* Formating

- Closes #37

* Close issue automatically

* Organization

* Fix

* #10

* ...

* Add label

* #10

* Adopt Get-Manifest funciton in Test-Hash

* Last touch on #10?

* Fix param rename

* Add hash extraction test for PR #36

* Idiot ... 🤦‍♀️🤦‍♀️

* Add only when there are hash properties #36

* Readme sync with new changes to flow

* Wording

* Prepare dockerfile for release

* Polish

* Rename tests
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request high-priority
Projects
None yet
Development

No branches or pull requests

1 participant