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

fix: remove unused parameter for LoggingWebMvcConfigurer. #3573

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

zhumin8
Copy link
Contributor

@zhumin8 zhumin8 commented Feb 21, 2025

We'd love to accept your patches and contributions to this project. There are just a few small
guidelines you need to follow before opening an issue or a PR:

  1. Ensure the issue was not already reported.
  2. Open a new issue if you are unable to find an existing issue addressing your problem. Make sure
    to include a title and clear description, as much relevant information as possible, and a code
    sample or an executable test case demonstrating the expected behavior that is not occurring.
  3. Discuss the priority and potential solutions with the maintainers in the issue. The maintainers
    would review the issue and add a label "Accepting Contributions" once the issue is ready for
    accepting contributions.
  4. Open a PR only if the issue is labeled with "Accepting Contributions", ensure the PR description
    clearly describes the problem and solution. Note that an open PR without an issue labeled with "
    Accepting Contributions" will not be accepted.

See
also CONTRIBUTING.md
.

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.

1 participant