Excel-DNA / Excel-DNA/ExcelDna

Review extended registration type support

Open
#726 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
C#
Stars
1.5k
Forks
292
Avg merge
20d 12h
Merged PRs (30d)
1

Description

We're working on expanding the default function registration and merging the ExcelDna.Registration library into the main add-in library. The goal is for a larger set of functions to 'just work' without requiring extra registration steps.
Since we are no longer targeting older .NET Framework versions (< .NET Framework 4.5) we can incorporate `Task` and other types into the main library.

Some of the current improvements are listed here: https://excel-dna.net/docs/guides-basic/extended-registration

I'm not sure what the right set of default type conversions are and realize that we're making a compatibility problem by returning object handles for all 'unknown' types.

I suggest we:
* Add support for `string[]` and `string[,]` return types, to match the parameter types
* Consider again whether the 'object handles' should catch all unknown types automatically. The problem is that once we return object handles for all unknown types, we cannot add automatic conversions for these types in future without breaking compatibility. Maybe we should reconsider whether a function should be explicitly marked to return / accept a handle?

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.