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.

Sign up for GitHub

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

Jump to bottom

Create workflowcheck.yml using the forked debugger repo and then showing off I can do this as a second PR using a workflow YML #9

Open
wants to merge 12 commits into
base: main
Choose a base branch
from
Open

Create workflowcheck.yml using the forked debugger repo and then showing off I can do this as a second PR using a workflow YML #9

wants to merge 12 commits into from

Conversation

bearycool11
Copy link

YAML skills check, CHECKED

@bearycool11
Create workflowcheck.yml
d34d20d 
YAML skills check, CHECKED

Signed-off-by: J. K. Edwards <[email protected]>
bearycool11
bearycool11 commented Jan 12, 2025
View reviewed changes
Copy link
Author

@bearycool11 bearycool11 left a comment

Choose a reason for hiding this comment

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

yes, this is indeed a YAML workflow protocol that will create checks for this PR

@bearycool11
Update workflowcheck.yml
70829b3 
trigger the next checks of the YAML workflow.

Signed-off-by: J. K. Edwards <[email protected]>
bearycool11
bearycool11 commented Jan 12, 2025
View reviewed changes
Copy link
Author

@bearycool11 bearycool11 left a comment

Choose a reason for hiding this comment

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

committed a second commit to showcase combination of two YAML workflows as a singular workflow YAML that goes through iteration steps and checks.

@bearycool11
Update workflowcheck.yml
53144b9 
Showcasing all the workflow YAMLS as a singular YAML protocol.

Signed-off-by: J. K. Edwards <[email protected]>
bearycool11
bearycool11 commented Jan 12, 2025
View reviewed changes
.workflow/workflowcheck.yml
Copy link
Author

Choose a reason for hiding this comment

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

We're going to show how we can create a singular YAML that will go through all the necessary YAML checks before I assign myself as a reviewer.

bearycool11
bearycool11 commented Jan 12, 2025
View reviewed changes
Copy link
Author

@bearycool11 bearycool11 left a comment

Choose a reason for hiding this comment

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

left a comment as a way to prove I read my own commits and checked my own code.

@bearycool11
Update workflowcheck.yml
0168183 
During combination and refactorization by addition and combination for YML workflows, especially if you're using AI Agents to build YAML workflows, you must never delete any of the workflow artifacts, even if it's a bit redundant.

Signed-off-by: J. K. Edwards <[email protected]>
bearycool11
bearycool11 commented Jan 12, 2025
View reviewed changes
Copy link
Author

@bearycool11 bearycool11 left a comment

Choose a reason for hiding this comment

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

completed combination and refactorization of the YAML workflow protocol commits.

@bearycool11
Update workflowcheck.yml
1c87b6d 
creating a new branch to begin a new PR

Signed-off-by: J. K. Edwards <[email protected]>
@bearycool11
Update workflowcheck.yml
4c11e7c 
making sure no information in code in the YML was lost in translation.

Signed-off-by: J. K. Edwards <[email protected]>
@bearycool11
Copy link
Author

Merged and rebased using the a separate branch to test the forked repo first using the check sum YAML workflow protocol.

@bearycool11 bearycool11 marked this pull request as draft January 12, 2025 07:47
@bearycool11 bearycool11 marked this pull request as ready for review January 12, 2025 07:48
bearycool11
bearycool11 commented Jan 12, 2025
View reviewed changes
Copy link
Author

@bearycool11 bearycool11 left a comment

Choose a reason for hiding this comment

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

I have reviewed my own code, it's time for the secondary review to also review the workflow protocol.

@bearycool11
Create createchecks.yml
167c722 
here it is

Signed-off-by: J. K. Edwards <[email protected]>
bearycool11
bearycool11 commented Jan 12, 2025
View reviewed changes
Copy link
Author

@bearycool11 bearycool11 left a comment

Choose a reason for hiding this comment

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

okay, good to go to check myself with checks

@bearycool11
Copy link
Author

waiting for checks information to show and pass....

@bearycool11
Update workflowcheck.yml
7260ca1 
Signed-off-by: J. K. Edwards <[email protected]>
@bearycool11
Copy link
Author

I could write a YML workflows to update the ruleset for bearycool to have write access to the repo I suppose....

@bearycool11
Update createchecks.yml (#3)
3a2731c 
rulset update for bearycool11

Signed-off-by: J. K. Edwards <[email protected]>
@bearycool11 bearycool11 marked this pull request as draft January 12, 2025 08:13
@bearycool11
Copy link
Author

hmmmmmmmm....

#9
#10
#6
#5
#1
#2
#10

Alright, those are the different PRs ATM.

@bearycool11 bearycool11 marked this pull request as ready for review January 12, 2025 08:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@bearycool11