-
Notifications
You must be signed in to change notification settings - Fork 103
Add full support for banners identified in #202 #203
Comments
@claudijd Why don't we maintain a list of the type of banners we support somewhere? Maybe like a todo list, on a wiki page? |
@agaurav77 because these types will likely change frequently and I'd like to manage them in only one place. |
Additionally, there will always be edge cases, so I'd rather have specs for the uniqs |
Added more cases in #218 Remaining banners to include in our unit-tests include... "SSH-2.0-mpSSH_0.2.1", |
Remaining items... "SSH-2.0-mpSSH_0.2.1", |
Remaining items: "SSH-2.0-mpSSH_0.2.1", |
All of these are now supported as of #229 |
Add banners from #202
The text was updated successfully, but these errors were encountered: