Skip to content

Removing unused namespaces#22

Merged
stevegrunwell merged 1 commit into
assertwell:developfrom
open-source-contributions:remove_unused_namespaces
Dec 30, 2020
Merged

Removing unused namespaces#22
stevegrunwell merged 1 commit into
assertwell:developfrom
open-source-contributions:remove_unused_namespaces

Conversation

@peter279k

@peter279k peter279k commented Dec 23, 2020

Copy link
Copy Markdown
Contributor

Changed log

  • Removing some namespaces because they're unused.
  • Using the message pattern to ignore the call_user_func_array parameter message.
  • Adding the @var array<string> for document annotation to fix the static code analysis.

stevegrunwell
stevegrunwell approved these changes Dec 30, 2020
View reviewed changes

@stevegrunwell stevegrunwell left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you for your contribution!

@stevegrunwell stevegrunwell merged commit 510f0ab into assertwell:develop Dec 30, 2020
@peter279k peter279k deleted the remove_unused_namespaces branch December 30, 2020 03:10
stevegrunwell added a commit that referenced this pull request Apr 15, 2021
@stevegrunwell
Ensure that #22 is included in v0.2.1
a28c743
@stevegrunwell stevegrunwell mentioned this pull request Apr 15, 2021
Merged
@stevegrunwell stevegrunwell added this to the Version 0.2.1 milestone Apr 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@stevegrunwell stevegrunwell stevegrunwell approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

Version 0.2.1

Development

Successfully merging this pull request may close these issues.

2 participants

@peter279k @stevegrunwell