Skip to content

Syncing from upstream faif/python-patterns (master) - #25

Merged
bt-admin merged 7 commits into
brain-tec:masterfrom
faif:master
Oct 18, 2025
Merged

Syncing from upstream faif/python-patterns (master)#25
bt-admin merged 7 commits into
brain-tec:masterfrom
faif:master

Conversation

@bt-admin

Copy link
Copy Markdown

bt_gitbot

Sorrow-Scarlet and others added 7 commits October 17, 2025 15:30
Use ellipsis replace pass to avoid potential warning:
```
	"resource": "/d:/Github/Python/python-patterns/patterns/creational/factory.py",
	"owner": "pylance4",
		"value": "reportReturnType",
			"path": "/microsoft/pylance-release/blob/main/docs/diagnostics/reportReturnType.md",
	"severity": 8,
	"message": "Function with declared return type \"str\" must return value on all code paths\n  \"None\" is not assignable to \"str\"",
	"source": "Pylance",
	"startLineNumber": 29,
	"startColumn": 37,
	"endLineNumber": 29,
	"endColumn": 40,
	"origin": "extHost1"
}]
```

Using VS Code with Pylance, Python debugger, and Black Formatter.
* Remove pluggable-libs submodule and cleanup (#433)

* Fix GitHub Actions output format error for multiline file lists

* Add requirements-dev.txt so CI can install dev dependencies
* Remove pluggable-libs submodule and cleanup (#433)

* Fix GitHub Actions output format error for multiline file lists

* Add requirements-dev.txt so CI can install dev dependencies

---------

Co-authored-by: Sakis Kasampalis <faif@users.noreply.github.com>
Co-authored-by: Benjamin Townsend <germin8@jamminangels.net>
Co-authored-by: Sakis Kasampalis <faif@users.noreply.github.com>
* Adding typehint

* Update __exit__ type

* fix black formatting

---------

Co-authored-by: Roy, Debakar <debakar.roy@intel.com>
Co-authored-by: Sakis Kasampalis <faif@users.noreply.github.com>
@bt-admin
bt-admin merged commit 1d6f2e4 into brain-tec:master Oct 18, 2025
3 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants