Skip to content

Conversation

@GangGreenTemperTatum
Copy link
Contributor

@GangGreenTemperTatum GangGreenTemperTatum commented Nov 25, 2024

using rigging <3 in a simple CICD workflow example

optional: add to the examples directory and|or add in some documentation somewhere

example garbage repo i tested and validated working for both PR descriptions and comments here, seemed only sensical to use PR description for this use-case and is most common

needs OPENAI_API_KEY GHA secret as per here which is the chosen generator for the custom class in this example

during testing, i did hard-pin pydantic (pip install pydantic==2.9.1) here but im confident this should be fixed from the recent release

if its good to go, i'll start adding it to some other public repos, especially will check with Simon if he is game for robopages etc

@GangGreenTemperTatum
Copy link
Contributor Author

using rigging <3 in a simple CICD workflow example

optional: add to the examples directory and|or add in some documentation somewhere

example garbage repo i tested and validated working for both PR descriptions and comments here, seemed only sensical to use PR description for this use-case and is most common

needs OPENAI_API_KEY GHA secret as per here which is the chosen generator for the custom class in this example

during testing, i did hard-pin pydantic (pip install pydantic==2.9.1) here but im confident this should be fixed from the recent release

if its good to go, i'll start adding it to some other public repos, especially will check with Simon if he is game for robopages etc

@GangGreenTemperTatum
Copy link
Contributor Author

@GangGreenTemperTatum
Copy link
Contributor Author

i also published this local commit as a follow on from upstream commit 1d12095 to fix the pydantic errors i was seeing and hence had hard-pinned beforehand

Copy link
Contributor

@monoxgas monoxgas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All looks awesome! Should be fun to play with :)

@GangGreenTemperTatum GangGreenTemperTatum removed the request for review from briangreunke December 6, 2024 19:19
@GangGreenTemperTatum
Copy link
Contributor Author

GangGreenTemperTatum commented Dec 6, 2024

All looks awesome! Should be fun to play with :)

thank you so much! :) could you please add the OPENAI_API_KEY GHA secret when you have a sec? so i can test this all works before merging

@GangGreenTemperTatum
Copy link
Contributor Author

All looks awesome! Should be fun to play with :)

thank you so much! :) could you please add the OPENAI_API_KEY GHA secret when you have a sec? so i can test this all works before merging

mb, apologies i'm a dummy! 🙈 it's because this PR comes from my fork
i'll merge it, removed some confusion in the PR template and it'll work on PRs from a clone of the repo (ie dreadnode peeps)

@GangGreenTemperTatum GangGreenTemperTatum merged commit 82ac80f into dreadnode:main Dec 7, 2024
3 of 4 checks passed
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.

2 participants