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 repository URL for Fiber Swagger #1233

Merged
merged 1 commit into from Jun 27, 2022
Merged

Conversation

ansidev
Copy link
Contributor

@ansidev ansidev commented Jun 16, 2022

Describe the PR
Update repository URL for Fiber Swagger (the repository was moved there).

The repository https://github.com/arsmn/fiber-swagger is deprecated.

@ubogdan
Copy link
Contributor

ubogdan commented Jun 18, 2022

What about https://github.com/swaggo/fiber-swagger ?

@ansidev
Copy link
Contributor Author

ansidev commented Jun 27, 2022

@ubogdan I believe this is a change which easy to understand.

The old URL https://github.com/arsmn/fiber-swagger is deprecated and moved to the new repository. This information was noted on the repository homepage. Did you visit them before commenting?

@ubogdan
Copy link
Contributor

ubogdan commented Jun 27, 2022

@ansidev no I didn't visit it. I expect to see a link to an issue or a well-documented PR.

Copy link
Contributor

@ubogdan ubogdan left a comment

Choose a reason for hiding this comment

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

LGTM

@ubogdan ubogdan merged commit 8335df9 into swaggo:master Jun 27, 2022
@ubogdan
Copy link
Contributor

ubogdan commented Jun 27, 2022

@ansidev Thanks for your contribution.

@ansidev
Copy link
Contributor Author

ansidev commented Jun 27, 2022

@ansidev no I didn't visit it. I expect to see a link to an issue or a well-documented PR.

@ubogdan I explained them in the PR description. I never doubt that it is clear until I read your comment.

Describe the PR Update repository URL for Fiber Swagger (the repository was moved there).

The repository arsmn/fiber-swagger is deprecated.

@ansidev ansidev deleted the patch-1 branch June 27, 2022 16:02
@ubogdan
Copy link
Contributor

ubogdan commented Jun 27, 2022

@ansidev I see.

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.

None yet

2 participants