Skip to content

IOS integration#39

Open
lil-jon-crunk wants to merge 4 commits intoAvionBlock:masterfrom
lil-jon-crunk:master
Open

IOS integration#39
lil-jon-crunk wants to merge 4 commits intoAvionBlock:masterfrom
lil-jon-crunk:master

Conversation

@lil-jon-crunk
Copy link

No description provided.

Copy link
Member

@SineVector241 SineVector241 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall very good. Just some version changes needed. Not sure why you also included nuget package metatada in the csproj and kept the original nuspec as a result but IG that's fine.

@@ -0,0 +1,6 @@
<Project>
<PropertyGroup>
<OpusSharpVersion>1.7.0</OpusSharpVersion>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Versions need to be 1.6.1. since the official opus version is currently 1.6.1.

<metadata>
<id>OpusSharp.Natives</id>
<version>1.6.0.1</version>
<version>1.7.0</version>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be 1.6.0.2

<metadata>
<id>OpusSharp</id>
<version>1.6.1</version>
<version>1.7.0</version>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be 1.6.0.2

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants