Skip to content

Deprecation changes and use of @impl in Plug Lesson#3074

Merged
erickgnavar merged 3 commits intoelixirschool:mainfrom
Jeberlen:patch-1
Jan 30, 2026
Merged

Deprecation changes and use of @impl in Plug Lesson#3074
erickgnavar merged 3 commits intoelixirschool:mainfrom
Jeberlen:patch-1

Conversation

@Jeberlen
Copy link
Contributor

Changes 'use Plug.Test' to 'import Plug.Test' as previous statement is deprecated

Changed 'defp handle_errors/2' to 'def handle_errors/2' with @impl Plug.ErrorHandler per https://hexdocs.pm/plug/Plug.ErrorHandler.html

Changes 'use Plug.Test' to 'import Plug.Test' as previous statement is deprecated. 

Changes 'defp handle_errors/2' to 
@impl Plug.ErrorHandler
def handle_errors/2
per https://hexdocs.pm/plug/Plug.ErrorHandler.html
@Jeberlen Jeberlen requested a review from a team as a code owner January 29, 2026 20:31
@doomspork
Copy link
Member

Thank you @Jeberlen! Would you be willing to find-and-replace this change across all translations?

@Jeberlen
Copy link
Contributor Author

Thank you @Jeberlen! Would you be willing to find-and-replace this change across all translations?

Yes, I did not think about that! Will get to it asap.

@Jeberlen Jeberlen closed this Jan 29, 2026
@Jeberlen Jeberlen reopened this Jan 29, 2026
@Jeberlen Jeberlen requested review from a team as code owners January 30, 2026 01:14
@erickgnavar erickgnavar merged commit e67ba17 into elixirschool:main Jan 30, 2026
1 check passed
Copy link
Contributor

@Markushik Markushik left a comment

Choose a reason for hiding this comment

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

[ru] correct ⭐️

@doomspork
Copy link
Member

Amazing, thank you @Jeberlen!

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.

4 participants