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.

Sign up for GitHub

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

Jump to bottom

Replace logrus with slog in most of lib/srv/app #47808

Merged
merged 4 commits into from
Oct 24, 2024
Merged

Replace logrus with slog in most of lib/srv/app #47808

merged 4 commits into from
Oct 24, 2024

Conversation

marcoandredinis
Copy link
Contributor

No description provided.

@marcoandredinis marcoandredinis added no-changelog Indicates that a PR does not require a changelog entry backport/branch/v15 backport/branch/v16 labels Oct 22, 2024
@aws-amplify-us-west-2 AWS Amplify (us-west-2)
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-47808.d3pp5qlev8mo18.amplifyapp.com

strideynet
strideynet reviewed Oct 22, 2024
View reviewed changes
lib/srv/app/aws/handler.go Outdated Show resolved Hide resolved
lib/srv/app/aws/handler.go Outdated Show resolved Hide resolved
lib/srv/app/aws/handler.go Outdated Show resolved Hide resolved
lib/srv/app/gcp/handler.go Outdated Show resolved Hide resolved
rosstimothy
rosstimothy approved these changes Oct 23, 2024
View reviewed changes
lib/srv/app/aws/handler.go Outdated Show resolved Hide resolved
@public-teleport-github-review-bot public-teleport-github-review-bot bot removed the request for review from Joerger October 23, 2024 23:55
@marcoandredinis marcoandredinis added this pull request to the merge queue Oct 24, 2024
Merged via the queue into master with commit 71292d4 Oct 24, 2024
39 checks passed
@marcoandredinis marcoandredinis deleted the marco/slog_libsrvapp branch October 24, 2024 08:22
@public-teleport-github-review-bot
Copy link

@marcoandredinis See the table below for backport results.

Branch Result
branch/v15 Create PR
branch/v16 Create PR

This was referenced Oct 24, 2024
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rosstimothy rosstimothy rosstimothy approved these changes

@strideynet strideynet strideynet approved these changes

Assignees
No one assigned
Labels
application-access backport/branch/v15 backport/branch/v16 no-changelog Indicates that a PR does not require a changelog entry size/sm
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@marcoandredinis @strideynet @rosstimothy