Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Issue 4630/replace default gateway#5255

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Conversation

@mahdikhashan
Copy link
Contributor

  • This is abugfix
  • This is afeature
  • This is acode refactor
  • This is atest update
  • This is adocs update
  • This is ametadata update

For Bugs and Features; did you add new tests?

there are failing tests in#5080 which will be fixed and new tests will be added in case.

Motivation / Use-Case

fixes#4630

Breaking Changes

Additional Info

@mahdikhashanmahdikhashan self-assigned thisAug 2, 2024
@mahdikhashanmahdikhashan marked this pull request as ready for reviewAugust 2, 2024 14:58
@alexander-akait
Copy link
Member

Please remove unrelated changes, thank you

mahdikhashan reacted with thumbs up emoji

@mahdikhashanmahdikhashanforce-pushed theissue-4630/replace-default-gateway branch fromdab3db4 tod744c9eCompareAugust 2, 2024 15:40
@mahdikhashanmahdikhashanforce-pushed theissue-4630/replace-default-gateway branch fromd744c9e tob5784a1CompareAugust 2, 2024 15:41
@mahdikhashan
Copy link
ContributorAuthor

Please remove unrelated changes, thank you

done.

alexander-akait reacted with heart emoji

*@param {string} gateway
*@returns {string | undefined}
*/
staticfindIp(gateway){

Choose a reason for hiding this comment

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

We need to keep this function, because developer still can use it, otherwise it is a breaking change

Copy link
ContributorAuthor

Choose a reason for hiding this comment

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

ok, I'll change my changes.

alexander-akait reacted with thumbs up emoji
Copy link
ContributorAuthor

Choose a reason for hiding this comment

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

done

@codecov
Copy link

codecovbot commentedAug 6, 2024
edited
Loading

Codecov Report

Attention: Patch coverage is82.75862% with5 lines in your changes missing coverage. Please review.

Project coverage is 84.99%. Comparing base(af6bd68) to head(f0ce3e8).
Report is 74 commits behind head on master.

FilesPatch %Lines
lib/Server.js82.75%4 Missing and 1 partial⚠️
Additional details and impacted files
@@            Coverage Diff             @@##           master    #5255      +/-   ##==========================================- Coverage   90.29%   84.99%   -5.31%==========================================  Files          15       16       +1       Lines        1577     1693     +116       Branches      601      635      +34     ==========================================+ Hits         1424     1439      +15- Misses        140      227      +87- Partials       13       27      +14

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report?Share it here.

@alexander-akait
Copy link
Member

/cc@snitin315 Feel free to merge when CI will be green, locally all tests are passed

snitin315 reacted with thumbs up emoji

@snitin315snitin315 merged commitf5f0902 intowebpack:masterAug 7, 2024
hzy added a commit to hzy/lynx-stack that referenced this pull requestMar 6, 2025
hzy added a commit to hzy/lynx-stack that referenced this pull requestMar 6, 2025
colinaaa pushed a commit to lynx-family/lynx-stack that referenced this pull requestMar 7, 2025
This is a mirror ofwebpack/webpack-dev-server#5255Fixes#83<!--  Thank you for submitting a pull request!We appreciate the time and effort you have invested in making thesechanges. Please ensure that you provide enough information to allowothers to review your pull request.Upon submission, your pull request will be automatically assigned withreviewers.If you want to learn more about contributing to this project, pleasevisit:https://github.com/lynx-family/lynx-stack/blob/main/CONTRIBUTING.md.-->## Summary<!-- Can you explain the reasoning behind implementing this change? Whatproblem or issue does this pull request resolve? --><!-- It would be helpful if you could provide any relevant context, suchas GitHub issues or related discussions. -->## Checklist<!--- Check and mark with an "x" -->- [ ] Tests updated (or not required).- [ ] Documentation updated (or not required).
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@snitin315snitin315snitin315 approved these changes

@hiroppyhiroppyAwaiting requested review from hiroppy

@anshumanvanshumanvAwaiting requested review from anshumanv

@alexander-akaitalexander-akaitAwaiting requested review from alexander-akait

Assignees

@mahdikhashanmahdikhashan

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Unusable IPv6 address printed

3 participants

@mahdikhashan@alexander-akait@snitin315

[8]ページ先頭

©2009-2025 Movatter.jp