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

Update README.md #316

Merged
merged 1 commit into from
Jan 4, 2024
Merged

Update README.md #316

merged 1 commit into from
Jan 4, 2024

Conversation

kevincobain2000
Copy link
Member

No description provided.

@kevincobain2000 kevincobain2000 merged commit 3fff810 into master Jan 4, 2024
14 checks passed
@kevincobain2000 kevincobain2000 deleted the kevincobain2000-patch-1 branch January 4, 2024 20:32
@@ -166,7 +166,7 @@ You write extra params with rules with @LRDparam in the comment line as one line
* // either space or pipe
* @LRDparam nickaname string|nullable|max:32
* // override the default response codes
* @LRDparam responses 200,422
* @LRDparam responses 200|422
Copy link
Contributor

Choose a reason for hiding this comment

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

This is actually wrong. It should be @LRDresponses, not @LRDparam

Copy link
Contributor

Choose a reason for hiding this comment

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

I'll make a pull request and update the readme with the changes from #314 explained too

Copy link
Contributor

Choose a reason for hiding this comment

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

here you go: #318

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