Skip to content
This repository has been archived by the owner on Apr 27, 2024. It is now read-only.

Update Readme #28

Merged
merged 3 commits into from
May 30, 2020
Merged

Update Readme #28

merged 3 commits into from
May 30, 2020

Conversation

RishabAgarwal
Copy link

Added Description till isD*. #23

Copy link
Collaborator

@parasg1999 parasg1999 left a comment

Choose a reason for hiding this comment

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

The table boundaries need to be corrected.

@RishabAgarwal
Copy link
Author

The table boundaries need to be corrected.

@parasg1999 could you explain what exactly to do?

@parasg1999
Copy link
Collaborator

@RishabAgarwal Seems like you've forgotten to add | for table boundaries

@parasg1999 parasg1999 added 🧹 needs-update For PRs that need to be updated before landing documentation Improvements or additions to documentation labels May 30, 2020
@RishabAgarwal RishabAgarwal requested a review from parasg1999 May 30, 2020 08:54
Copy link
Member

@rubiin rubiin 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

@rubiin rubiin merged commit b703603 into validatorjs:master May 30, 2020
@rubiin
Copy link
Member

rubiin commented May 30, 2020

you can now go along with other documentation if you like

@rubiin
Copy link
Member

rubiin commented May 30, 2020

also be sure to ask to change the status of documentation done in the tracking issue

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🧹 needs-update For PRs that need to be updated before landing documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants