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-109276: regrtest: shorter list of resources by vstinner · Pull Request #110326 · python/cpython Conversation
|
|
Examples: |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
@serhiy-storchaka: Does it look better like that? |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
I did not have problems with the older output, but if you want to change it, the new one looks better.
Did you consider the idea of making it compatible with the input to the -u option? I.e. remove space after comma and remove plus sign. Alternatively you can make -u accepting input with spaces and pluses (although copying text with spaces is less convenient). Not that I personally need this feature, but it might come in handy one day.
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Done.
Done. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
| for name in sorted(all_resources - use_resources): | ||
| relative_all.append(f'-{name}') | ||
| for name in sorted(use_resources - all_resources): | ||
| relative_all.append(f'{name}') |
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
| relative_all.append(f'{name}') | |
| relative_all.append(name) |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
Oh right. Sadly, I saw your comment after the change was merged. I make try to include it in my next regrtest change ;-)
Sorry, something went wrong.
There was an error while loading. Please reload this page.
For me, it's not easy to know if it's default --slow-ci option, default --fast-ci option, or if -u options were modified on a buildbot. With the new output relative to |
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.