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

docs(homepage): discord flat badge style; remove former devax email #3768

Merged
merged 1 commit into from Feb 14, 2024

Conversation

hjgraca
Copy link
Contributor

@hjgraca hjgraca commented Feb 14, 2024

Issue number: #3769

Summary

Updates to readme

Changes

  • Style Discord badge
  • Remove DevAx email

User experience

image

Checklist

If your change doesn't seem to apply, please leave them unchecked.

Is this a breaking change?

RFC issue number:

Checklist:

  • Migration process documented
  • Implement warnings (if it can live side by side)

Acknowledgment

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

Signed-off-by: Henrique Graca <999396+hjgraca@users.noreply.github.com>
@hjgraca hjgraca requested a review from a team as a code owner February 14, 2024 15:48
@pull-request-size pull-request-size bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Feb 14, 2024
Copy link
Contributor

No related issues found. Please ensure there is an open issue related to this change to avoid significant delays or closure.

@github-actions github-actions bot added do-not-merge need-issue PRs that are missing related issues labels Feb 14, 2024
Copy link

sonarcloud bot commented Feb 14, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues

Measures
0 Security Hotspots
No data about Coverage
1.8% Duplication on New Code

See analysis details on SonarCloud

@heitorlessa
Copy link
Contributor

thank you @hjgraca for the proactiveness! We need an issue before we can merge.

@heitorlessa heitorlessa changed the title chore: Update README - style badge. Remove email docs(homepage): discord flat badge style; remove former devax email Feb 14, 2024
@heitorlessa heitorlessa removed do-not-merge need-issue PRs that are missing related issues labels Feb 14, 2024
@heitorlessa
Copy link
Contributor

Thank you @hjgraca for the issue :) It helps us take the same path customers and contributors take (dogfooding), plus make it traceable.

I've adjusted the title to have a better changelog page. Merging!

docs(homepage): discord flat badge style; remove former devax email

@heitorlessa heitorlessa merged commit ea47f53 into aws-powertools:develop Feb 14, 2024
13 checks passed
@hjgraca hjgraca deleted the patch-1 branch February 14, 2024 16:47
@rubenfonseca rubenfonseca linked an issue Feb 19, 2024 that may be closed by this pull request
2 tasks
@heitorlessa heitorlessa added the documentation Improvements or additions to documentation label Feb 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Maintenance: Update readme to style badge and remove email
2 participants