-
Notifications
You must be signed in to change notification settings - Fork 1
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
Automatization of labels in GitHub Actions #11
Comments
@Erick11293 How is it going? Shall we wrap up this by the end of the week? Thanks!! |
Oh actually github is much more efficient than slack. |
I completely agree. I'll work on this at the end of the day. Tomorrow, I'll get an update. |
Perfect! Love you 🐍 Let me know when you need me to test this new automation on my end. |
I am wondering apart from what we already updated, can we let the workflow itself to identify whether this is a blank repo. If this is a blank repo, we can push directly. If not, we can do as what we have now in Github Action, clone, checkout and push. |
@ShiqiYang2022 I had the same idea, and I tried to do that on my test on Monday. Sadly, I think github actions are not allowed to create branches, or at least not using the code we have your instructions. We can explore this option if we have time. |
@Erick11293 I think I have complete the upgrading Wiki, see here, to make it clearer to implement. Do you have time to do a test tomorrow on the Automation part strictly follow the new instructions? I think 1 hour at most would be enough. Thanks! |
Per conversation, @jc-cisneros will test the newly updated automation process using his template. Ideally it will work(and it should work!). Feel free to put comments here(informal comment is very acceptable and welcomed!). Ideally we shall not spend no more than 1 hour. Thanks chief! 👑 |
@ShiqiYang2022 per our conversations in Slack, now the process is working in both directions. Great work! Some minor points:
|
@jc-cisneros Thanks for your time to test! I agree 100% we should make the setup process more clear and it's exactly the purpose of testing. I have updated the new instructions here. If you have time could you please take a look at it? |
Close this issue as completed. Summary: Final delivarble can be found here |
Follow the gentzkow#84 (comment).
The text was updated successfully, but these errors were encountered: