Skip to content

Upgrade fog-aliyun to 0.3.10#1606

Merged
aashah merged 1 commit intocloudfoundry:masterfrom
xiaozhu36:alicloud
May 11, 2020
Merged

Upgrade fog-aliyun to 0.3.10#1606
aashah merged 1 commit intocloudfoundry:masterfrom
xiaozhu36:alicloud

Conversation

@xiaozhu36
Copy link
Copy Markdown
Contributor

@xiaozhu36 xiaozhu36 commented May 7, 2020

The fog-aliyun 0.3.10 aims to fix the issue: fog/fog-aliyun#61 and exlarge the limitation when getting oss objects. The related PRs as following:
fog/fog-aliyun#65
fog/fog-aliyun#64
fog/fog-aliyun#63
fog/fog-aliyun#62

  • I have reviewed the contributing guide

  • I have viewed, signed, and submitted the Contributor License Agreement

  • I have made this pull request to the master branch

  • I have run all the unit tests using bundle exec rake

  • I have run CF Acceptance Tests

  • I have deploy a whole CF deployment and spring-music app by packing gem fog-aliyun 0.3.10 to Capi-release.

@cf-gitbot
Copy link
Copy Markdown

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/172713125

The labels on this github issue will be updated when the story is started.

@xiaozhu36 xiaozhu36 changed the title Upgrade fog-aliyun to 0.3.9 Upgrade fog-aliyun to 0.3.10 May 7, 2020
@aashah
Copy link
Copy Markdown
Contributor

aashah commented May 9, 2020

Hey @xiaozhu36,

Looks like a super easy PR to merge, thanks! Quick clarification -- title says 0.3.10, commit says 0.3.9. Should we hold off from merging this until it's updated for 0.3.10?

@xiaozhu36
Copy link
Copy Markdown
Contributor Author

HI @aashah Thanks for your review. I am sorry for the confused commit message and I have corrected it. Please check it again. Thanks.

@aashah aashah merged commit b2848da into cloudfoundry:master May 11, 2020
@aashah
Copy link
Copy Markdown
Contributor

aashah commented May 11, 2020

No worries, thanks for updating the PR.

Will merge this in then.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants