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

GitHub actions #3

Closed
wants to merge 7 commits into from
Closed

GitHub actions #3

wants to merge 7 commits into from

Conversation

MatthiasLohr
Copy link
Contributor

This is a first draft of GitHub actions utilization. I never worked with GitHub actions before, so I'm not completely sure about the process of releasing/uploading artifacts to PyPI.

@MatthiasLohr MatthiasLohr mentioned this pull request Mar 2, 2021
@cburgdorf
Copy link
Collaborator

Hey @MatthiasLohr, that looks good! Thanks for setting it up. I can probably take it from here to wire up the rest of the pipeline that pushes the releases to pypi. But actually, I'll need your help here @davesque. I don't have administrative rights in this repository to setup secrets which we need to do to push into PyPi.

I see three possible ways forward and all of them are fine to me:

  1. You take it from here and wire up the rest to push releases to pypi. You remain the only one with administrative rights.

  2. You grant me administrative rights and I'll set it up (creating a new pypi token from my username to be used exclusively for this project)

  3. We fork this into an official ethereum repository and take care of the rest.

1, 2 and 3 are ordered from most hassle on your end to most hassle on my end 😂 What would you prefer?

@MatthiasLohr
Copy link
Contributor Author

I know I'm not eligible to vote... but I vote for option 3! (also considering the comment from @fubuloubu in #2). Btw, instead of forking, you could also "transfer" it to Ethereum group - so it would look more like the "main" repository. I usually check twice before I create a pull request - and mostly tend to open it again the "fork head" instead of some forks.

@cburgdorf
Copy link
Collaborator

Yeah, I can see why option 3 might be compelling but I also don't want to "steal" it if David wants to continue to maintain it. And thanks for the transfer suggestion, makes sense 👍

@MatthiasLohr
Copy link
Contributor Author

Any update on this...?

@cburgdorf
Copy link
Collaborator

@davesque Would you be ok to transfer ownership to the EF? I'm going to also send you an email just in case you aren't observing these messages here.

@davesque
Copy link
Contributor

davesque commented Mar 9, 2021

@cburgdorf Hey, yeah I would be happy to transfer ownership. Sorry I haven't been paying much attention to this repo or the issues on it. Isn't there some process I have to follow for that? I seem to remember that I did something similar for the rust-vyper (now fe) repo and had some interactions with @holiman regarding that. Maybe that was in the ethereum meta issues repo.

@cburgdorf
Copy link
Collaborator

cburgdorf commented Mar 10, 2021

Hey, yeah I would be happy to transfer ownership.

Thanks, happy to hear!

It sounds like that you would need to have rights to create repositories in the EF organization to be allowed to make the transfer. So maybe someone from devops needs to be added as an owner to this repo first.

image

The transfer has the blessing from @pipermerriam. I'll create an issue in the EFs devops system to see who can assist us with the transfer.

@MatthiasLohr
Copy link
Contributor Author

Hey, any progress here?

@cburgdorf
Copy link
Collaborator

Hey, any progress here?

Sorry, holdup were on my part. I missed the response from devops.

@davesque do you mind making @Aldekein from devops an admin on this repo so he can initiate the transfer?

@davesque
Copy link
Contributor

@cburgdorf Alright, there was no option to make @Aldekein an admin since this is a personal repo. So I just transferred the repo to him. I think he can finish the process of transferring it to the EF.

@cburgdorf
Copy link
Collaborator

Thanks, that should work 👍

@cburgdorf
Copy link
Collaborator

Migration worked, thanks! I still need to get access to the management of the environment variables exposed to Github Actions to complete the process but we should soon be able to wrap this up :)

@MatthiasLohr
Copy link
Contributor Author

Hi, sorry to bother you again, but I really would like to see that fixed... ;)

@cburgdorf
Copy link
Collaborator

Yep, me too. I pinged devops again today because I'm still lacking the administrative rights to setup the pypi token. I'll keep pushing this week :)

@Aldekein
Copy link
Member

@cburgdorf now you have administrative rights for the blake2b-py repo. Sorry for the delay!

@cburgdorf
Copy link
Collaborator

@Aldekein Thanks, everything is working fine now.

@MatthiasLohr This now landed as part of #4

I also cut a new version 0.1.4 wich also covers Python 3.9. You can see the packages on pypi. Thanks for pushing this forward :)

@cburgdorf cburgdorf closed this Mar 22, 2021
@MatthiasLohr
Copy link
Contributor Author

Finally!!! Thanks :) And sorry for pushing so hard ;)

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

Successfully merging this pull request may close these issues.

4 participants