Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Scaffold] TableFromExpression and QueryFromExpression support #4506

Open
wants to merge 1 commit into
base: refactor/parsing
Choose a base branch
from

Conversation

MaceWindu
Copy link
Contributor

@MaceWindu MaceWindu commented May 13, 2024

  • updated CLI scaffold to use TableFromExpression and QueryFromExpression helper for generated table functions
  • updated T4 scaffold to use TableFromExpression helper for generated table functions
  • removed CLI naming options table-function-methodinfo-field-name
  • [Firebird] updated boolean type scaffold to use DataType.Boolean
  • broke .cctor call loop between provider detectors and DataConnection
  • [T4][Access] removed T4 function LoadAccessMetadataByProvider, use LoadAccessMetadata instead
  • [T4][Informix] removed T4 function LoadInformixMetadata with string providerName overload. Use same function without additional parameter
  • [T4][Informix] added InformixProvider provider optional parameter to LoadInformixMetadata
  • [T4][SqlServer] remove provider optional parameter from LoadSqlServerMetadata API
  • [T4][SAP HANA] removed string providerName optional parameter in LoadSapHanaMetadata API

As a bonus, .NET FX tests work again in VS

@MaceWindu
Copy link
Contributor Author

/azp run test-all

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@MaceWindu
Copy link
Contributor Author

/azp run test-all

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@MaceWindu
Copy link
Contributor Author

/azp run test-all

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@linq2dbot
Copy link

Test baselines changed by this PR. Don't forget to merge/close baselines PR after this pr merged/closed.

@MaceWindu
Copy link
Contributor Author

/azp run test-all

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@MaceWindu
Copy link
Contributor Author

/azp run test-all

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@MaceWindu
Copy link
Contributor Author

/azp run test-all

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@linq2dbot
Copy link

Test baselines changed by this PR. Don't forget to merge/close baselines PR after this pr merged/closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants