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

Integrate all-green #1632

Closed
wants to merge 45 commits into from
Closed

Integrate all-green #1632

wants to merge 45 commits into from

Conversation

x1101
Copy link
Contributor

@x1101 x1101 commented Jun 22, 2024

Added All-green github action to ci.yaml, requiring the nox job

x1101 and others added 30 commits May 15, 2024 00:17
Markdown standards fixup

Co-authored-by: Don Naro <dnaro@redhat.com>
wording fixup

Co-authored-by: Don Naro <dnaro@redhat.com>
Clearer description for why/when to tag

Co-authored-by: Don Naro <dnaro@redhat.com>
language clarity fixup

Co-authored-by: Don Naro <dnaro@redhat.com>
Co-authored-by: Don Naro <dnaro@redhat.com>
Co-authored-by: Don Naro <dnaro@redhat.com>
silly typo fixup

Co-authored-by: Don Naro <dnaro@redhat.com>
clarity fixup

Co-authored-by: Don Naro <dnaro@redhat.com>
Co-authored-by: Don Naro <dnaro@redhat.com>
typo

Co-authored-by: Don Naro <dnaro@redhat.com>
Co-authored-by: Don Naro <dnaro@redhat.com>
Co-authored-by: Don Naro <dnaro@redhat.com>
Co-authored-by: Don Naro <dnaro@redhat.com>
Co-authored-by: Don Naro <dnaro@redhat.com>
Co-authored-by: Don Naro <dnaro@redhat.com>
Co-authored-by: Don Naro <dnaro@redhat.com>
Co-authored-by: Sandra McCann <samccann@redhat.com>
Co-authored-by: Sandra McCann <samccann@redhat.com>
@x1101
Copy link
Contributor Author

x1101 commented Jun 22, 2024

This is related to issue #1284

@oraNod oraNod requested a review from webknjaz June 24, 2024 08:45
with:
allowed-failures: docs, linters
allowed-skips: non-voting-flaky-job
jobs: ${{ toJSON(needs) }}
Copy link
Contributor

Choose a reason for hiding this comment

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

@x1101 Just taking a cursory look through for now. I'll take another look tomorrow. Could you please add an extra line at the end of this file and the other one?

@oraNod
Copy link
Contributor

oraNod commented Jun 24, 2024

@x1101 Thanks for taking this on. It looks like you're working off the devel branch of your fork instead of a feature branch. I see quite a few commits that are unrelated to the changes in this PR too.

If I may make a suggestion, try creating a new feature branch with a descriptive name that tracks the devel branch of ansible/ansible-documentation and work off that branch. You can rebase on devel and keep your work more logically organized that way. Please feel free to ask if you have any questions. Cheers.

@x1101 x1101 closed this by deleting the head repository Jun 24, 2024
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.

None yet

2 participants