Inconsist type annotation in build_schedule_from_dbt_selection

[tmastny]

What's the issue?

The tags parameter is annotated as tags: Optional[Mapping[str, str]] = None,

dagster/python_modules/libraries/dagster-dbt/dagster_dbt/asset_utils.py

Lines 276 to 286 in cf951d6

	def build_schedule_from_dbt_selection(
	dbt_assets: Sequence[AssetsDefinition],
	job_name: str,
	cron_schedule: str,
	dbt_select: str = "fqn:*",
	dbt_exclude: Optional[str] = None,
	schedule_name: Optional[str] = None,
	tags: Optional[Mapping[str, str]] = None,
	config: Optional[RunConfig] = None,
	execution_timezone: Optional[str] = None,
	default_status: DefaultScheduleStatus = DefaultScheduleStatus.STOPPED,

This parameter is immediately forwarded to define_asset_job:

dagster/python_modules/libraries/dagster-dbt/dagster_dbt/asset_utils.py

Lines 328 to 336 in cf951d6

	job=define_asset_job(
	name=job_name,
	selection=build_dbt_asset_selection(
	dbt_assets,
	dbt_select=dbt_select,
	dbt_exclude=dbt_exclude,
	),
	config=config,
	tags=tags,

However, the tags parameter there is annotated as tags: Optional[Mapping[str, object]] = None,

dagster/python_modules/dagster/dagster/_core/definitions/unresolved_asset_job_definition.py

Lines 247 to 255 in f46f14f

	def define_asset_job(
	name: str,
	selection: Optional["CoercibleToAssetSelection"] = None,
	config: Optional[
	Union[ConfigMapping, Mapping[str, Any], "PartitionedConfig", "RunConfig"]
	] = None,
	description: Optional[str] = None,
	tags: Optional[Mapping[str, object]] = None,
	run_tags: Optional[Mapping[str, object]] = None,

What did you expect to happen?

I expected the type annotation to match.

How to reproduce?

No response

Dagster version

Copy/pasted from master.

Deployment type

None

Deployment details

No response

Additional information

No response

Message from the maintainers

Impacted by this issue? Give it a 👍! We factor engagement into prioritization.