Skip to content

Commit f80e870

Browse files
committed
Remove unresolved question
1 parent ef67a8c commit f80e870

File tree

1 file changed

+1
-25
lines changed

1 file changed

+1
-25
lines changed

rfcs/0002-feature-flags.md

+1-25
Original file line numberDiff line numberDiff line change
@@ -62,28 +62,4 @@ Currently, `WebAsyncWrapper` is a separate class that is only compatible with Wi
6262

6363
# Unresolved questions
6464

65-
Should the public API stay the same across features?
66-
67-
Approach A, add methods, properties, etc. to public API:
68-
69-
```vb
70-
' Method isn't even available without feature
71-
#If EnableFeatureA Then
72-
Public Function DoA()
73-
74-
End Function
75-
#End If
76-
```
77-
78-
Approach B, enable methods, properties, etc.:
79-
80-
```vb
81-
' Method is disabled without feature
82-
Public Function DoA()
83-
#If EnableFeatureA Then
84-
85-
#End If
86-
End Function
87-
```
88-
89-
Approach A would lead to compiler errors if a method/property is used when the flag is turned off, which I think is a good thing since it would allow errors to be caught earlier.
65+
None.

0 commit comments

Comments
 (0)