Skip to content

Updated contribution guidelines link #13013

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
merged 2 commits into from
Aug 19, 2025

Conversation

VanshikaSabharwal
Copy link
Contributor

Motivation

Fixes:- #12907
The Contributing Guidelines link shown in issues is currently pointing to
https://github.com/localstack/.github/blob/main/CONTRIBUTING.md (incorrect).
This confuses contributors since the actual contributing guide for LocalStack lives at
https://github.com/localstack/localstack/blob/main/docs/CONTRIBUTING.md.

Changes

  • Updated .github/ISSUE_TEMPLATE/config.yml so that the "Contributing Guidelines" contact link points to the correct file inside the localstack/localstack repository.

Steps To Reproduce (before fix)

  1. Go to https://github.com/localstack/localstack/issues
  2. Open any listed issue
  3. Check the "Contributing Guidelines" link added by localstack-bot
  4. It incorrectly points to .github/blob/main/CONTRIBUTING.md

Expected Behavior (after fix)

The "Contributing Guidelines" link should point to:
https://github.com/localstack/localstack/blob/main/docs/CONTRIBUTING.md

Environment

  • OS: Any
  • LocalStack: Any
  • How started: With a docker run command (not relevant for this bug)

Anything else?

N/A

Copy link
Contributor

@localstack-bot localstack-bot left a comment

Choose a reason for hiding this comment

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

Welcome to LocalStack! Thanks for raising your first Pull Request and landing in your contributions. Our team will reach out with any reviews or feedbacks that we have shortly. We recommend joining our Slack Community and share your PR on the #community channel to share your contributions with us. Please make sure you are following our contributing guidelines and our Code of Conduct.

@localstack-bot
Copy link
Contributor

localstack-bot commented Aug 17, 2025

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

Copy link
Member

@alexrashed alexrashed left a comment

Choose a reason for hiding this comment

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

Thanks a lot for the contribution, @VanshikaSabharwal! You are absolutely right, the link to the org-wide contribution guides is confusing here in this repo, given that we have quite extensive repo-specific guidelines here.
Your change didn't quite hit the issue you explained (the bot message), but it's a good point to also add the guidelines to the issue template.
I pushed a small commit (126c26b) to adjust the bot message you have been referring to.

Now the PR is looking good to be merged! Could you please sign our CLA as described in this comment (just post the mentioned sentence without anything else as a comment to this PR)?
Thanks!

@alexrashed alexrashed self-assigned this Aug 18, 2025
@alexrashed alexrashed added the semver: patch Non-breaking changes which can be included in patch releases label Aug 18, 2025
@alexrashed alexrashed added this to the 4.8 milestone Aug 18, 2025
@VanshikaSabharwal
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

@VanshikaSabharwal
Copy link
Contributor Author

Thanks a lot for the contribution, @VanshikaSabharwal! You are absolutely right, the link to the org-wide contribution guides is confusing here in this repo, given that we have quite extensive repo-specific guidelines here. Your change didn't quite hit the issue you explained (the bot message), but it's a good point to also add the guidelines to the issue template. I pushed a small commit (126c26b) to adjust the bot message you have been referring to.

Now the PR is looking good to be merged! Could you please sign our CLA as described in this comment (just post the mentioned sentence without anything else as a comment to this PR)? Thanks!

Yes. I am a new contributor to LocalStack, so I was finding it difficult to find the code for the bot, and thank you so much for helping me out. I am looking forward to contributing more and learn a lot.

@alexrashed alexrashed merged commit f323916 into localstack:main Aug 19, 2025
5 of 7 checks passed
@alexrashed
Copy link
Member

Awesome! Thanks a lot, congrats to your first contribution, and we are looking forward to your next one! 🤩

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver: patch Non-breaking changes which can be included in patch releases
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants