GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | ||||||||||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | ||||||||||
GeezerVison is showing github.com — gh-105983: add incognito mode to webbrowser module by AmirNakhaeii · Pull Request #106069 · python/cpython Conversation
|
|
Most changes to Python require a NEWS entry. Please add it using the blurb_it web app or the blurb command-line tool. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
|
Duplicate of #105984 |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
@Eclips4 |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
|
good day @AmirNakhaeii, just received your message via email. when you would please have a look at the history of the initial pull request #105984 and the code given there you will see that i already did the development before i have created the issue #105983 regarding the new feature. it is a bit confusing sometimes how github organize the flow of information. anyway i saw that you added also support for edge. thats great because i dont have microsoft windows. probably best would be that you wait until my pull request is merged, and then you create an own issue regarding the enhancement and support for edge. and an extra pull request for that. myself would prefer this approach since i can not spent time for taking part in a development where i dont have the proper OS and can not even test it. thanks for your contribution, and understanding. ps. do close your pull request #106069, also because you followed the naming convention and gave it the same name as mine - thats might be confusing to others - |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
just corrected the different pull requests numbers in my former reply. sorry for the confusion. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
i dont know if it is possible to create "depening" issues in github. but probably it might be also an idea to clone my git repo and checkout the branch https://github.com/kr-g/cpython/tree/kr-g-browser-incognito and create from this HEAD ref a new branch where you put your code for edge browser. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
please have also a look here https://github.com/python/cpython/pull/105984/files at the test cases -> Lib/test/test_webbrowser.py you need to create also one for Edge. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
@kr-g |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
@kr-g |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
ok fine, i try to keep that in mind |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Successfully merging this pull request may close these issues.
There was an error while loading. Please reload this page.