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

Replace the usual CI with a comment based on the branch #19

Closed
wants to merge 4 commits into from

Conversation

liamhuber
Copy link
Member

I have no intent of merging this, I just want to play with the idea on a public repo analogous to @mbruns91 graphathon

Copy link

Binder 👈 Launch a binder notebook on branch pyiron/pyiron_snippets/ci_experiment

At some point the extra namespace `rest` got inserted in the call path, I forget when but I've dealt with this update before
Copy link

Copy link

@liamhuber
Copy link
Member Author

Hmm, seems like the CI is stuck with the version in the initial push? I still got a comment, but it's the "less sophisticated" one from an earlier commit. Let's try turning it off and turning it on again!

@liamhuber liamhuber closed this Jun 19, 2024
@liamhuber liamhuber reopened this Jun 19, 2024
Copy link

@liamhuber
Copy link
Member Author

Oh, derp, no, it's because I am not passing the actual comment to the API call 🤦‍♂️

Copy link

🤖 Here is the source code for the example notebook on this branch

@liamhuber
Copy link
Member Author

Mission accomplished, cool.

@liamhuber liamhuber closed this Jun 19, 2024
@liamhuber liamhuber deleted the ci_experiment branch June 19, 2024 16:49
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.

1 participant