Skip to content

add license badges to the readme file #1732

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Jun 1, 2024
Merged

add license badges to the readme file #1732

merged 2 commits into from
Jun 1, 2024

Conversation

@alguerocode
Copy link
Contributor

@alguerocode alguerocode commented Sep 20, 2021 •
edited
Loading

add license tag or badges to the readme file

Checklist

  • README updated (where applicable)

alguerocode
alguerocode commented Sep 20, 2021
View reviewed changes
@codecov
Copy link

codecov bot commented Sep 20, 2021 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (c25b98f) to head (04b09e3).
Report is 3 commits behind head on master.

❗ Current head 04b09e3 differs from pull request most recent head 247f175

Please upload reports for the commit 247f175 to get more accurate results.

Additional details and impacted files 
@@            Coverage Diff             @@
##            master     #1732    +/-   ##
==========================================
  Coverage   100.00%   100.00%            
==========================================
  Files          109       101     -8     
  Lines         2489      2005   -484     
  Branches       628       452   -176     
==========================================
- Hits          2489      2005   -484

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@alguerocode alguerocode changed the title add license badges add license badges to the readme file Sep 20, 2021
tux-tn
tux-tn previously approved these changes Sep 21, 2021
View reviewed changes
Copy link
Member

@tux-tn tux-tn left a comment

Choose a reason for hiding this comment

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

LGTM ! Thank you @alguerocode for your contribution

@alguerocode
Copy link
Contributor Author

alguerocode commented Sep 21, 2021

thank you to bro

@alguerocode
Copy link
Contributor Author

alguerocode commented Sep 29, 2021

🥇 👍

ezkemboi
ezkemboi previously approved these changes Sep 29, 2021
View reviewed changes
@ezkemboi
Copy link
Member

ezkemboi commented Sep 29, 2021

This will be merged soon @alguerocode

@rubiin rubiin added mc-to-land Just merge-conflict standing between the PR and landing. and removed needs-more-review labels May 12, 2024
@rubiin rubiin dismissed stale reviews from ezkemboi and tux-tn via 247f175 June 1, 2024 18:58
@rubiin rubiin removed the mc-to-land Just merge-conflict standing between the PR and landing. label Jun 1, 2024
@rubiin rubiin requested a review from profnandaa June 1, 2024 18:59
@rubiin rubiin added the ✅ LGTM label Jun 1, 2024
@profnandaa profnandaa merged commit 21f33b6 into validatorjs:master Jun 1, 2024
@WikiRik WikiRik mentioned this pull request Mar 20, 2025
Merged
@fossabot fossabot bot mentioned this pull request Oct 28, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@profnandaa profnandaa profnandaa approved these changes

@ezkemboi ezkemboi ezkemboi left review comments

@tux-tn tux-tn Awaiting requested review from tux-tn

Assignees

No one assigned

Labels

🎉 first-pr ✅ LGTM

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@alguerocode @ezkemboi @profnandaa @tux-tn @rubiin