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

Use ansible-documentation repo for core porting guide #540

Merged
merged 1 commit into from
Jul 18, 2023

Conversation

gotmax23
Copy link
Contributor

As of ansible-community/community-topics#240, the ansible documentation (including porting guides) are stored in a new repository.

@gotmax23 gotmax23 requested a review from felixfontein July 18, 2023 15:47
@gotmax23
Copy link
Contributor Author

@samccann FYI

@gotmax23
Copy link
Contributor Author

Force pushed to fix formatting errors 🙈

Copy link
Collaborator

@felixfontein felixfontein left a comment

Choose a reason for hiding this comment

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

Thanks for working on this! This needs a changelog fragment.

src/antsibull/changelog.py Show resolved Hide resolved
src/antsibull/constants.py Outdated Show resolved Hide resolved
Copy link
Contributor Author

@gotmax23 gotmax23 left a comment

Choose a reason for hiding this comment

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

I'll add the changelog fragment, and then I think we should merge and cut a bugfix release. I think it's important that artifacts we create are (somewhat) reproducible using the current antsibull release. Let's create a follow up issue about fixing the tag references.

src/antsibull/changelog.py Show resolved Hide resolved
src/antsibull/constants.py Outdated Show resolved Hide resolved
@gotmax23
Copy link
Contributor Author

Force pushed to add a changelog fragment

As of ansible-community/community-topics#240,
the ansible documentation (including porting guides) are stored in a new
repository.
@felixfontein felixfontein merged commit 1af3c9f into ansible-community:main Jul 18, 2023
@felixfontein
Copy link
Collaborator

@gotmax23 thanks for fixing this! If you have time, can you do a new bugfix release?

@gotmax23
Copy link
Contributor Author

Yeah, I was planning on it later today.

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