Skip to content
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

fix: Dependency Requiring CGO #67

Closed
wants to merge 9 commits into from
Closed

Conversation

bbernays
Copy link
Contributor

@bbernays bbernays commented Apr 18, 2022

I switched from Azure/azure-storage-blob-go/azblob to Azure/azure-sdk-for-go/sdk/storage/azblob for the azblob as azure-storage-blob-go has been deprecated.

Used the unreleased version gocloud.dev package:

@hairyhenderson
Copy link
Owner

Oh thank you! I guess this will fix #64.

Not sure why CI isn't kicking off... 🤔

Signed-off-by: Dave Henderson <dhenderson@gmail.com>
@hairyhenderson
Copy link
Owner

@bbernays can you rebase? I've fixed the CI configuration in #68 and if you rebase it should kick off correctly...

@bbernays
Copy link
Contributor Author

@hairyhenderson - I have rebased, but it seems like I need a maintainer to approve running workflows

@hairyhenderson
Copy link
Owner

Looks like the rebase didn't work quite right - there shouldn't need to be a merge commit in there...

Alternately, can you enable the Allow edits by maintainers checkbox?

image

@hairyhenderson
Copy link
Owner

(in the meantime, I have approved the workflows - CI's running now, which is progress! 😉)

commit afb9d05
Merge: 37e50d6 7b6420d
Author: Dave Henderson <dhenderson@gmail.com>
Date:   Tue Apr 19 13:21:07 2022 -0400

    Merge pull request hairyhenderson#70 from hairyhenderson/dependabot/github_actions/actions/checkout-3

    Bump actions/checkout from 2 to 3

commit 7b6420d
Author: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Date:   Tue Apr 19 16:28:20 2022 +0000

    Bump actions/checkout from 2 to 3

    Bumps [actions/checkout](https://github.com/actions/checkout) from 2 to 3.
    - [Release notes](https://github.com/actions/checkout/releases)
    - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
    - [Commits](actions/checkout@v2...v3)

    ---
    updated-dependencies:
    - dependency-name: actions/checkout
      dependency-type: direct:production
      update-type: version-update:semver-major
    ...

    Signed-off-by: dependabot[bot] <support@github.com>
@hairyhenderson
Copy link
Owner

@bbernays I'm still seeing commits that shouldn't be here...

Can you do:

$ git fetch upstream
$ git rebase upstream/main
$ git push -f

@bbernays
Copy link
Contributor Author

I am closing this pr, because I opened a new clean PR

@bbernays bbernays closed this Apr 19, 2022
@hairyhenderson
Copy link
Owner

ok - this one was salvageable if you'd done a git rebase - but a new PR is fine 🙂

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

Successfully merging this pull request may close these issues.

2 participants