Skip to content

Clarified the instructions in issue templates #52

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

Merged

Conversation

SableRaf
Copy link
Contributor

Added clearer instructions about required files (e.g., .txt and .zip) with examples for proper hosting, and link to the Guidelines to help contributors meet requirements more easily.

Copy link

netlify bot commented Jan 13, 2025

Deploy Preview for processing-contributions ready!

Name Link
🔨 Latest commit 9eb36b3
🔍 Latest deploy log https://app.netlify.com/sites/processing-contributions/deploys/67858279d28e3a0008b00c4c
😎 Deploy Preview https://deploy-preview-52--processing-contributions.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@SableRaf SableRaf requested review from mingness and Stefterv January 13, 2025 19:33
@SableRaf SableRaf changed the title Improved the documentation for issue templates Clarified the instructions in issue templates Jan 13, 2025
Copy link
Collaborator

@mingness mingness left a comment

Choose a reason for hiding this comment

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

Looks good. I wonder if a github release url would be a useful example, instead of the mywebsite.com example?

Copy link
Collaborator

@mingness mingness left a comment

Choose a reason for hiding this comment

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

You hint at the use of latest, which should not change from version to version, but maybe we should more explicitly say, that this url should not change version to version.

Copy link
Collaborator

@mingness mingness left a comment

Choose a reason for hiding this comment

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

looks great to me!

@SableRaf SableRaf merged commit 907f14c into processing:main Jan 13, 2025
4 checks passed
@SableRaf SableRaf deleted the sableraf-issue-tempate-better-instructions branch January 13, 2025 21:20
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