mirror of
https://github.com/yswery/PHP-DNS-SERVER.git
synced 2026-04-26 09:45:54 +03:00
[PR #62] [MERGED] Feature/version 1 additions #78
Labels
No labels
bug
enhancement
enhancement
hacktoberfest
pull-request
question
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/PHP-DNS-SERVER#78
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/yswery/PHP-DNS-SERVER/pull/62
Author: @ivanstan
Created: 9/11/2018
Status: ✅ Merged
Merged: 9/13/2018
Merged by: @samuelwilliams
Base:
version-1← Head:feature/version-1-additions📝 Commits (10+)
477990dResolved issues in Server class reported by phpstorm77f6c13Resolved issues in JsonResolverTest and JsonResolver class reported byca04d3eResolved issues in ClassEnum class reported byee53104Resolved issues in Decoder class reported by5973549Resolved issues in Encoder class reported byae311acResolved issues in Message class reported by86d69a9Resolved issues in RecordTypeEnum class reported bya20de87Resolved issues in RecursiveResolver class reported by6a561ddResolved issues in ResourceRecord class reported by12e44f7Resolved issues in ResourceRecord class reported by📊 Changes
13 files changed (+181 additions, -196 deletions)
View changed files
📝
README.md(+1 -3)📝
src/ClassEnum.php(+5 -5)📝
src/Decoder.php(+19 -19)📝
src/Encoder.php(+23 -21)📝
src/JsonResolver.php(+28 -30)📝
src/Message.php(+8 -8)📝
src/RecordTypeEnum.php(+47 -47)📝
src/RecursiveResolver.php(+11 -12)📝
src/ResourceRecord.php(+13 -13)📝
src/Server.php(+15 -27)📝
src/Tests/JsonResolverTest.php(+5 -5)📝
src/Tests/RecordTypeEnumTest.php(+5 -5)📝
src/Tests/ServerTest.php(+1 -1)📄 Description
I basically went trough classes and resolved whatever phpstorm inspections reported
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.