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

GitHub Discussions for questions and translations #1572

Merged
merged 18 commits into from Jan 9, 2024

Conversation

Zeitsperre
Copy link
Collaborator

@Zeitsperre Zeitsperre commented Dec 24, 2023

Pull Request Checklist:

  • This PR addresses an already opened issue (for bug fixes / features)
  • Tests for the changes have been added (for bug fixes / features)
    • (If applicable) Documentation has been added / updated (for bug fixes / features)
  • CHANGES.rst has been updated (with summary of main changes)
    • Link to issue (:issue:number) and pull request (:pull:number) has been added

What kind of change does this PR introduce?

  • Migrates the Question/Support Issue Template to a GitHub Discussion Template
  • Adds a new page for coordinating translation efforts
  • Added a link to PAVICS for PAVICS-specific questions. Once we have a Discourse up and running, we'll be able to funnel all PAVICS-specific question there.

Does this PR introduce a breaking change?

Not really. This makes it so that the following can happen more easily:

  • Answered support requests will more easily be found by users with similar usage questions. This will make it simpler for users as answering a discussion post does not hide the post like closing an issue does.
  • Translation efforts can now more easily be coordinated outside of Pull Requests for users wishing to contribute translations in languages other than English or French. In a previous attempt, the major translation effort was performed in a Pull Request with 100+ comments in it and was impossible to navigate near the end. I've noticed similar translation efforts (using GitHub Discussions) organized in other projects, such as FastAPI (https://github.com/tiangolo/fastapi/discussions/categories/translations).

Other information:

https://docs.github.com/en/discussions/managing-discussions-for-your-community/creating-discussion-category-forms

@Zeitsperre Zeitsperre added standards / conventions Suggestions on ways forward information For development/intsructional purposes support Questions and help for users/developers labels Dec 24, 2023
@Zeitsperre Zeitsperre added this to the v0.48.0 milestone Dec 24, 2023
@Zeitsperre Zeitsperre requested a review from a team December 24, 2023 19:45
@Zeitsperre Zeitsperre self-assigned this Dec 24, 2023
@github-actions github-actions bot removed the information For development/intsructional purposes label Dec 24, 2023
@bzah
Copy link
Contributor

bzah commented Dec 26, 2023

I would suggest to also update the Readme, in particular the Contributing to xclim section in order to guide user to the github discussion forum instead of gitter.

@Zeitsperre
Copy link
Collaborator Author

I would suggest to also update the Readme, in particular the Contributing to xclim section in order to guide user to the github discussion forum instead of gitter.

That's a good idea. I don't seem to have admin access over the Gitter room, so I can't make changes. Does anyone know who created the room? @huard?

@github-actions github-actions bot added the docs Improvements to documenation label Dec 29, 2023
@github-actions github-actions bot added the information For development/intsructional purposes label Dec 29, 2023
@Zeitsperre
Copy link
Collaborator Author

Update: so in order to retire the Gitter room, I need to manually kick everyone out of it (and ask any existing moderators to leave). It doesn't seem like there's an easy way to close down a room other than booting everyone and making it private.

I've saved the chat log (it wasn't very long, TBF), so once this Pull Request is in, I'll start the process of winding the Gitter down.

.github/DISCUSSION_TEMPLATE/questions.yml Outdated Show resolved Hide resolved
@github-actions github-actions bot added the approved Approved for additional tests label Jan 8, 2024
@coveralls
Copy link

coveralls commented Jan 9, 2024

Coverage Status

coverage: 90.304%. remained the same
when pulling db440a7 on discussions-templates
into dd9db13 on master.

@Zeitsperre Zeitsperre merged commit bc41a1c into master Jan 9, 2024
15 of 16 checks passed
@Zeitsperre Zeitsperre deleted the discussions-templates branch January 9, 2024 23:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Approved for additional tests docs Improvements to documenation information For development/intsructional purposes standards / conventions Suggestions on ways forward support Questions and help for users/developers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Adopt Category Forms for Community Questions
4 participants