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

links.yml: remove IRC and ML #312

Closed

Conversation

Andersson007
Copy link
Collaborator

SUMMARY

links.yml: remove IRC and ML

Copy link

github-actions bot commented Sep 3, 2024

Docs Build 📝

This PR is closed and any previously published docsite has been unpublished.

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.

Adding a negative review since https://forum.ansible.com/t/6812 hasn't been decided yet.

@Andersson007
Copy link
Collaborator Author

Andersson007 commented Sep 3, 2024

Adding a negative review since https://forum.ansible.com/t/6812 hasn't been decided yet.

@felixfontein what is exactly meant by deciding?
This is "default" and not mandatory if maintainers disagree.
I don't think it should be a mandatory policy of sort and IRC or whatever else to get officially forbidden.
If the maintainers/communities of specific projects still want to use IRC, I think it's up to them and they can close my PRs.

Maybe I misunderstood the intention, @gundalow ? If we are going to forbid IRC and make the forum mandatory at policy level within community projects I don't know how to handle this and whether it's worth doing at all:)

@gundalow
Copy link
Contributor

gundalow commented Sep 3, 2024

Let's use the forum discussion

@Andersson007
Copy link
Collaborator Author

Let's use the forum discussion

SGTM, moved my comment there. There was already a comment by Felix expressing concerns, so the order feels logical:) PTAL

@felixfontein
Copy link
Collaborator

#314 removes the mailing list link.

@Andersson007
Copy link
Collaborator Author

@felixfontein rebased, PTAL

@Andersson007
Copy link
Collaborator Author

@felixfontein the poll results are ready.
Looks like the community is OK with removal of IRC whatsoever or keeping refs to active rooms. Anyway, it's up to maintainers of particular projects to decide what to use, so please merge or close the PR. Thanks

@felixfontein
Copy link
Collaborator

I think for now, let's keep the IRC reference. Thanks for your work on this!

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.

3 participants